Skip to content

Conversation

@YASH-SAWARKAR
Copy link

Problem

  • Email composer was missing a FROM field to select sender email
  • Users couldn't choose which configured email account to send from

Solution

  • ✅ Added FROM field to email composer interface
  • ✅ Created SingleSelectEmailInput component for single email selection
  • ✅ Fetches available email accounts from Email Account doctype (with enable_outgoing: 1)
  • ✅ Auto-selects first available email account by default

Let’s say we have two email accounts added: [email protected] and [email protected] . Now we can manually select sender email while composing a new mail or replying.

Closes : #796 #1078

Before:
image

After:
image

image

@aquiveal
Copy link

You have no idea how timely this is! I was literally planning to work on this exact feature this weekend. Thank you so much for this PR!

@YASH-SAWARKAR
Copy link
Author

You're welcome, @aquiveal. Good to hear that it helped you.

@YASH-SAWARKAR
Copy link
Author

@shariquerik What are your thoughts on this? Does it need any changes?

@shariquerik
Copy link
Member

@YASH-SAWARKAR You are getting outgoing emails list which is not valid because there can be multiple outgoing emails but we should only show logged in user's emails check User doctype there is an
Screenshot 2025-08-28 at 12 39 10 PM

You can check how this feature is implemented in desk for reference.

@federicocalvo
Copy link

@shariquerik I don't know how they were using it, if this is a bug that appeared now as described here (#1194) but this directly limits the most important use of the CRM, to be able to send mails having more than one account configured in the ERP, and the only way it works, is placing as default account the account you use in the CRM.

Unless there is something I'm missing I see several people interested in this PR maybe because of this.

@federicocalvo
Copy link

@YASH-SAWARKAR any news about this PR? it is hard for me to understand how someone uses the crm with the ERP, if it forces you to make the email account the default outgoing account.

@YASH-SAWARKAR
Copy link
Author

@federicocalvo I DM'd some queries to @shariquerik and I’m waiting for a reply. BTW, if someone has configured emails, they should be able to select the ‘From’ email of their choice (it solves this PR).

@aquiveal
Copy link

aquiveal commented Sep 2, 2025

@YASH-SAWARKAR The issue @shariquerik suggested earlier was that the user was given access to everyone's and all email accounts, which is not okay because any member could use each other's email accounts.

@aquiveal
Copy link

aquiveal commented Sep 9, 2025

@shariquerik Did you get a chance to check it out?

@federicocalvo
Copy link

Any new about this?

I think I have something configured incorrectly, because otherwise it would be impossible for them to be using the emails in the CRM if it allows you to use another account, which is the one designated as the default sender.

@YASH-SAWARKAR
Copy link
Author

I’ve updated PR #1264. @shariquerik , could you please review it?
cc: @aquiveal @federicocalvo

@federicocalvo
Copy link

I still don't understand why this issue isn't being prioritized, or if there's something we're not understanding.

To clarify, when you usually have an email address for notifications or other things, here you have to force it to be the default sending account, forcing you to use the same account you use in the CRM, such as the default notifications account.

For example:
Captura de pantalla 2025-09-24 a las 2 27 40 p  m

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.

Allow changing from while writing email

4 participants