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

Make reply button automatically fetch latest draft #308

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wheelie89
Copy link

Currently, when you click "Reply" whilst already having a draft in progress, it will create a new (empty) email.

Many mail clients do this differently, hitting the "Reply" button in Gmail for example will bring up the draft that you were previously composing.

This PR adds this functionality:

ProtonMail Reply Draft

I'm not sure what your policies regarding are, I'm doing this by passing down the conversationID, looking up the most recent draft via the new useLastDraft(), then passing that to onCompose.

If there is no previous draft, it will simply continue to do the "new empty message" thing that it's currently doing in production.

@wheelie89
Copy link
Author

Oh and I'm not sure what the implications on fromUndo are - something with keeping the password around but idk much about email encryption, please tell me if that bit should be changed.

@Guillaumez
Copy link
Contributor

Hello and thanks for your contribution !
This feature is not on our roadmap yet but discussions are ongoing about it.

Have a nice day !

@wheelie89
Copy link
Author

Is there any place where users can ask you guys regarding whether you are generally open for contributions, whether you are considering a feature etc.?

We are thinking about moving our inbox from gmail to proton but it does seem lacking in some regards.

@wheelie89
Copy link
Author

It's unfortunate there has been no further action to this. Our deadline is the following month in which we will probably settle with the paid gmail

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants