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

Commerce not using the From Name setting #939

Closed
nickcobley opened this issue Jul 25, 2019 · 5 comments
Closed

Commerce not using the From Name setting #939

nickcobley opened this issue Jul 25, 2019 · 5 comments
Labels
ℹ️ status: need more info When waiting for user to supply database or more information.

Comments

@nickcobley
Copy link

Description

I'm seeing an issue where the setting in Craft Commerce > General Settings > From Name, is not being used.

The notes for this field suggest it should be using the Site Name from General Settings, however, for me it's taking the Sender Name from Email Settings. However note I'm using the MailGun plugin, I'm not able to easily test right now if this problem goes away when not using a mail handler plugin.

For me this is identical to #535 just in case this is a regression

Steps to reproduce

  1. Change From Name setting
  2. Trigger status email

Additional info

  • Craft version: 3.2.6
  • Commerce: 2.1.10
  • PHP version: 7.3
  • Database driver & version: MySQL 5.7.25
  • Plugins & versions: Mailgun 1.4.2
@nickcobley nickcobley changed the title Commerce not using the From Name value Commerce not using the From Name setting Jul 26, 2019
@lukeholder
Copy link
Member

Can you try overriding the email too and let me know if it then picks up the override from name?

@lukeholder lukeholder added the ℹ️ status: need more info When waiting for user to supply database or more information. label Aug 1, 2019
@nickcobley
Copy link
Author

Sorry can you clarify that a bit more, do you mean try overriding the email address?

@lukeholder
Copy link
Member

lukeholder commented Aug 1, 2019

Try setting both the 'From Email' and the 'From Name'.

@nickcobley
Copy link
Author

Sorry for the late reply on this one, just found time to test it.

Your spot on, if you set a value in both fields it works.

nfourtythree added a commit that referenced this issue Aug 21, 2019
…e-address

Fixed #939 emails not using From Name
@nfourtythree
Copy link
Contributor

Hi @nickcobley

An update for this has been pushed and will be included in the next release.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ℹ️ status: need more info When waiting for user to supply database or more information.
Projects
None yet
Development

No branches or pull requests

3 participants