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

Module name in order details data #8483

Merged
merged 1 commit into from Nov 10, 2017

Conversation

Projects
None yet
3 participants
@kpodemski
Contributor

kpodemski commented Nov 8, 2017

Questions Answers
Branch? develop
Description? i just had a situation where i needed it directly in order-confirmation.tpl :)
Type? improvement
Category? CO
BC breaks? no
Deprecations? no
Fixed ticket? no
How to test? obvious

Important guidelines


This change is Reviewable

@kpodemski

This comment has been minimized.

Show comment
Hide comment
@kpodemski

kpodemski Nov 8, 2017

Contributor

btw. maybe you could guys add it to 1.7.3? very little change

Contributor

kpodemski commented Nov 8, 2017

btw. maybe you could guys add it to 1.7.3? very little change

@mickaelandrieu

This comment has been minimized.

Show comment
Hide comment
@mickaelandrieu

mickaelandrieu Nov 9, 2017

Contributor

Sadly, it's too late for 1.7.3 :(

Contributor

mickaelandrieu commented Nov 9, 2017

Sadly, it's too late for 1.7.3 :(

@kpodemski

This comment has been minimized.

Show comment
Hide comment
@kpodemski

kpodemski Nov 9, 2017

Contributor

Ok, no problem :)

Contributor

kpodemski commented Nov 9, 2017

Ok, no problem :)

@Quetzacoalt91 Quetzacoalt91 added this to the 1.7.4.0 milestone Nov 10, 2017

@Quetzacoalt91 Quetzacoalt91 merged commit 3ced84d into PrestaShop:develop Nov 10, 2017

2 checks passed

codacy/pr Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Quetzacoalt91

This comment has been minimized.

Show comment
Hide comment
@Quetzacoalt91

Quetzacoalt91 Nov 10, 2017

Member

Here you go! Thank you @kpodemski

Member

Quetzacoalt91 commented Nov 10, 2017

Here you go! Thank you @kpodemski

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