-
Notifications
You must be signed in to change notification settings - Fork 5
feat(email): schedule sending emails #1119
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
base: main
Are you sure you want to change the base?
Conversation
…t optionLabel to return empty string
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
|
Personally, I'm not a fan of removing the compose box from the bottom of the thread view. It makes it so that you can't scroll up and read emails in the thread while you are writing a message. Also, running locally, when I open email threads there is no reply box at the bottom like you mentioned in the PR description |
Fair enough, I think I'll add the bottom compose box back and leave the decision to remove it to another time if it comes up as an issue. |
|
A common use case for scheduled send is to send something the next day, at the start of the business day (i.e. 9AM). I can't figure out how to set the date AND time I want a message to be sent with the current UI. |
I removed the time selector in one of my recent commits because it doesn't fit the rest of the theme of the app since it's the native selector and it isn't able to be styled. So maybe we can just ff this feature until time selection is added |
…c/macro into rahul/feat-email-scheduled-send
This PR adds the ability to schedule a message and modify or remove the scheduled time.
In addition:
BaseInputandComposeto indicate when an email is being saved and has been savedEmailContextwere returned as different entity "kinds"DateSelectorcomponentScreen.Recording.2026-01-23.at.3.07.41.PM.mov