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

Keep sending email if legacy lang file not found #6733

Merged
merged 1 commit into from Oct 20, 2016

Conversation

Projects
None yet
2 participants
@julienbourdeau
Contributor

julienbourdeau commented Oct 20, 2016

Questions Answers
Branch? develop
Description? Report of 951c35e
Type? bug fix
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket?
How to test?
@julienbourdeau

This comment has been minimized.

Show comment
Hide comment
@julienbourdeau

julienbourdeau Oct 20, 2016

Contributor

Red build is a false alam, we checked here locally.

Contributor

julienbourdeau commented Oct 20, 2016

Red build is a false alam, we checked here locally.

@julienbourdeau julienbourdeau merged commit 2931305 into PrestaShop:develop Oct 20, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@julienbourdeau julienbourdeau deleted the julienbourdeau:fix/not-found-email-subject branch Oct 20, 2016

@hadjedjvincent

This comment has been minimized.

Show comment
Hide comment
@hadjedjvincent

hadjedjvincent Oct 20, 2016

Contributor

It would be fine to log it instead of doing nothing there ? Should be applied to 1.6.x too

Contributor

hadjedjvincent commented on classes/Mail.php in 36db78f Oct 20, 2016

It would be fine to log it instead of doing nothing there ? Should be applied to 1.6.x too

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