Problem/Motivation

When sending placing, cancelling, and shipping orders in the Drupal Commerce admin UI triggers emails. However, because of the fact that the emails are triggered in the addmin UI, they are sent with the admin theme instead of the default theme of the domain.

Proposed resolution

I propose that the domain option for theme settings be reimplemented using domain_theme_switch.

Remaining tasks

- Configure the domain option to work with a current domain module.

User interface changes

- Add domain option in mailsystem theme settings

API changes

- None

Data model changes

- None

CommentFileSizeAuthor
#3 mailsystem-reimplement-domain-theme-option-2941723-2.patch3.35 KBqymanab

Comments

qymanab created an issue. See original summary.

qymanab’s picture

qymanab commented
Related issues: +#3014053: Provide a way to access a domain's default theme
qymanab’s picture

qymanab commented
StatusFileSize
new mailsystem-reimplement-domain-theme-option-2941723-2.patch3.35 KB
josephdpurcell’s picture

josephdpurcell commented
Status: Active » Reviewed & tested by the community

There are no tests and I didn't manually test this. I don't see code style violations and the patch applies cleanly to the latest branch.

Setting to RTBC to get visibility to this proposal. Note that it is dependent upon #3014053: Provide a way to access a domain's default theme.

qymanab’s picture

qymanab commented
Status: Reviewed & tested by the community » Needs work

The proposed changes in the most current patch do not represent the most recent changes to the related domain_theme_switch issue. The implementation needs to be updated to reflect those changes.