-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add BCC email to spree_store #3646
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@seand7565 thanks for adding this new feature, I think it's going to be useful to many stores 👍
nitpick: I noticed that in the screenshot the label BCC email
is not capitalized like the others, maybe that needs to be done with CSS?
Adds a BCC email field to the spree store edit page. This indicates a BCC email that you want all order confirmation emails to be BCCed to. As indicated in solidusio#3615, this is a feature that I think a lot of eCommerce stores are interested in, at the very least to server as a notification of a new order. Also adds the ability to add/update this field through the API
awesome! Thanks Sean. I think we had this present in the past as well, not sure why it got removed then, but this is super useful! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @peterberkenbosch for asking the question I was about to. 😂
Looks good to me.
Thanks @seand7565! |
I already had a PR for that feature, but it was refused because it was not enough general/flexible: #2905 |
THIS PR SPONSORED BY Super Good Software
Description
Adds a BCC email field to the spree store edit page.
This indicates a BCC email that you want all order confirmation
emails to be BCCed to. As indicated in #3615 , this is a feature
that I think a lot of eCommerce stores are interested in, at
the very least to server as a notification of a new order.
Also adds the ability to add/update this field through the API
Checklist: