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

GOCOMMERCE_SMTP_ADMIN_EMAIL (and GOCOMMERCE_MAILER_ADMIN_EMAIL) are ignored #127

Closed
pukhalski opened this issue Sep 11, 2018 · 7 comments
Closed

Comments

@pukhalski
Copy link

- Do you want to request a feature or report a bug?
Bug

- What is the current behavior?
Although both GOCOMMERCE_SMTP_ADMIN_EMAIL and GOCOMMERCE_MAILER_ADMIN_EMAIL are set, order confirmation emails go from no-reply@netlify.com no-reply@netlify.com

- If the current behavior is a bug, please provide the steps to reproduce.
Deploy gocommerce to Heroku, set GOCOMMERCE_SMTP_ADMIN_EMAIL and make an order.

- What is the expected behavior?
GOCOMMERCE_SMTP_ADMIN_EMAIL contains "From" field email./

@phae phae added the bug label Sep 14, 2018
@mraerino
Copy link
Member

mraerino commented Oct 24, 2018

To verify this issue I setup a GoCommerce single-tenant instance on Heroku and set GOCOMMERCE_SMTP_ADMIN_EMAIL to my custom email address.

When ordering I verified that these mails were sent:

  • Order confirmation to customer - sent from my admin email
  • Order notification to admin email - sent from my admin email

@pukhalski could you check if this is still happening for you and otherwise close this issue?

@bcomnes
Copy link
Contributor

bcomnes commented Dec 5, 2018

I dont see GOCOMMERCE_MAILER_ADMIN_EMAIL in the configuration types. Is it implemented?

I do see SMTP_ADMIN_EMAIL thought.

@mraerino
Copy link
Member

mraerino commented Dec 5, 2018

You are right. GOCOMMERCE_MAILER_ADMIN_EMAIL is not available.

But GOCOMMERCE_SMTP_ADMIN_EMAIL is the one that's supposed to work which I verified.

@bcomnes
Copy link
Contributor

bcomnes commented Dec 5, 2018

Are these settings that should be moved to the settings.json file rather than the addon instance configuration / ENV var configuration? @mraerino and or @biilmann perhaps you have insight here.

@mraerino
Copy link
Member

mraerino commented Dec 6, 2018

For Netlify Identity / GoTrue this is also kept in the instance config.
I don't see any reason why this should be configured in the shop site.

@bcomnes
Copy link
Contributor

bcomnes commented Dec 6, 2018

I think you're right.

@bcomnes
Copy link
Contributor

bcomnes commented Dec 6, 2018

@pukhalski Not sure where GOCOMMERCE_MAILER_ADMIN_EMAIL is coming from. Can you give me a pointer? Sounds like this issue is working with the ENV var so this should be good to close. If we have additional issues with the addons parameters, or the other MAILER_ADMIN_EMAIL setting we should open a separate issues.

@bcomnes bcomnes closed this as completed Dec 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants