Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possibility to attach local files [$50 awarded] #251

Closed
10 tasks
jancborchardt opened this issue Dec 17, 2016 · 4 comments
Closed
10 tasks

Possibility to attach local files [$50 awarded] #251

jancborchardt opened this issue Dec 17, 2016 · 4 comments

Comments

@jancborchardt
Copy link
Member

jancborchardt commented Dec 17, 2016

(Carrying over the tasks from #4 so we don’t forget it ;)

client-side TODO

  • upload files asynchronously
  • add uploaded files to list of attachments
  • show loading and/or progress indicator
  • lock 'Send'/'Reply' button until all local attachments have been uploaded successfully
  • handle errors (remove faulty attachment from list)
  • add special property to attachment so that server can distinguish between local and cloud attachments

server-side TODO

  • add attachment store for asynchronously uploaded files
  • move uploaded attachments to temp dir (use core's temp manager)
  • limit uploaded file size to a sane size (e.g. a few MBs)
  • distinguish between local and cloud attachments when sending a message and load local attachments from the attachment store

cc @nextcloud/mail


The $50 bounty on this issue has been claimed at Bountysource.

@rakshazi
Copy link

Hello!
Any updates here?

@jancborchardt
Copy link
Member Author

@rakshazi unfortunately not yet, no. Since you are a developer it would be awesome if you can contribute a part of this via pull request – any contribution is welcome and much appreciated! :) We need more people working on the Mail app.

@jospoortvliet jospoortvliet changed the title Possibility to attach local files Possibility to attach local files [$50] Jul 4, 2017
@ChristophWurst
Copy link
Member

@lcalaresu FYI put a bounty on this long awaited feature 🤑

@jospoortvliet jospoortvliet changed the title Possibility to attach local files [$50] Possibility to attach local files [$50 awarded] Mar 7, 2018
@lock
Copy link

lock bot commented Nov 20, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and questions.

@lock lock bot locked and limited conversation to collaborators Nov 20, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants