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

Fixed MailsProvider service declaration name #11151

Merged
merged 1 commit into from Oct 29, 2018

Conversation

Projects
None yet
3 participants
@mickaelandrieu
Contributor

mickaelandrieu commented Oct 25, 2018

Questions Answers
Branch? develop
Description? The service name was inconsistent
Type? bug fix
Category? CO
BC breaks? no
Deprecations? no
How to test? Tests should pass

This change is Reviewable

@mickaelandrieu mickaelandrieu force-pushed the mickaelandrieu:fix-mails-provider-service-name branch from a8d3dc7 to 5ac9a0c Oct 25, 2018

@matks

matks approved these changes Oct 29, 2018

@mickaelandrieu mickaelandrieu merged commit 217e54c into PrestaShop:develop Oct 29, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@mickaelandrieu mickaelandrieu deleted the mickaelandrieu:fix-mails-provider-service-name branch Oct 29, 2018

@matks matks added this to the 1.7.6.0 milestone Nov 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment