Skip to content
This repository has been archived by the owner on Nov 8, 2018. It is now read-only.

return UID correctly so old drafts are automatically removed #1464

Merged
merged 1 commit into from
May 3, 2016

Conversation

ChristophWurst
Copy link
Contributor

@ChristophWurst ChristophWurst commented Apr 29, 2016

fixes #1450

@jancborchardt please review

@jancborchardt
Copy link
Contributor

Might fix the multiple drafts problem while writing a new message. But editing a draft still keeps the old one and creates a new thing:
capture du 2016-05-02 17-16-52

@ChristophWurst
Copy link
Contributor Author

I guess you mean the following workflow:

  1. start writing a new message, stop, wait so the draft is saved to the server
  2. open the drafts folder
  3. open the newly created draft
  4. resume writing
  5. another draft is saved instead of replacing the previous one

@ChristophWurst
Copy link
Contributor Author

In any case, there was an issue with that workflow as the UID of the original draft was not sent when updating it, hence it was not 'replaced'. Should be good now. Let me know if there are any other major issues with drafts, thanks.

@jancborchardt
Copy link
Contributor

Okidoke, thanks a lot! 👍

@jancborchardt jancborchardt merged commit 6de0d6f into master May 3, 2016
@jancborchardt jancborchardt deleted the fix-drafts-issues branch May 3, 2016 16:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants