Skip to content

[v3.2] Do not require 'mail'#4792

Merged
waiting-for-dev merged 1 commit intov3.2from
backport/v3.2/pr-4740
Dec 20, 2022
Merged

[v3.2] Do not require 'mail'#4792
waiting-for-dev merged 1 commit intov3.2from
backport/v3.2/pr-4740

Conversation

@github-actions
Copy link
Copy Markdown

@github-actions github-actions Bot commented Dec 19, 2022

Backport

This will backport the following commits from master to v3.2:

Questions ?

Please refer to the Backport tool documentation

@tvdeyen
Do not require 'mail'
6795597 
This was added a long time ago and is actually not necessary
anymore. If a shop depends on mail then through actionmailer.

Refs #4734

(cherry picked from commit 4ff8593)
@github-actions github-actions Bot mentioned this pull request Dec 19, 2022
Merged
3 tasks
@waiting-for-dev waiting-for-dev changed the title [v3.2] Merge pull request #4740 from tvdeyen/remove-mail-require [v3.2] Do not require 'mail' Dec 20, 2022
@waiting-for-dev waiting-for-dev merged commit 938ff3c into v3.2 Dec 20, 2022
@waiting-for-dev waiting-for-dev deleted the backport/v3.2/pr-4740 branch December 20, 2022 17:12
@waiting-for-dev waiting-for-dev added the changelog:solidus_core Changes to the solidus_core gem label Dec 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kennyadsl kennyadsl kennyadsl approved these changes

Assignees

@tvdeyen tvdeyen

Labels

changelog:solidus_core Changes to the solidus_core gem

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kennyadsl @tvdeyen @waiting-for-dev