-
Notifications
You must be signed in to change notification settings - Fork 5
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
Only save Draft when adding Attachment #1832
Conversation
feb9794
to
58d270f
Compare
34f4e48
to
ba4a4c5
Compare
58d270f
to
b6a6d35
Compare
ba4a4c5
to
62ac7e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This display a toast (should be a snackbar I think as we stay in the app) when coming b ack to the activity after saving an attachment
This is annoying as we are already displaying a toast when quitting the activity to open the picker
We probably shouldn't display anything when coming back to the activity
EDIT: it will be done, but in another PR
Pull request was converted to draft
ab72feb
to
0b1c0bc
Compare
998a574
to
8f5b285
Compare
0b1c0bc
to
a00c96b
Compare
6740f33
to
5be7da0
Compare
9d40271
to
5be7da0
Compare
Please retry analysis of this Pull-Request directly on SonarCloud |
We were saving Draft everytime we add or remove an Attachment, it's too much.
Now, we only save Draft when adding an Attachment.
Also, if we add a lot of Attachments at the same time (via the FilePicker), and we delete them all while it's still saving, the DraftsActionsWorker will fail because it's won't find the files on the phone disk.
Now, it will just ignore these failures so the Draft can be saved.