Skip to content

Conversation

@coderimus
Copy link
Contributor

@coderimus coderimus commented Mar 9, 2018

Description

Hello,
I found not using variables in price templates and also made small refactoring for using customer URL in the order view info template :)

Fixed Issues (if relevant)

N/A

Manual testing scenarios

N/A

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@coderimus
Copy link
Contributor Author

Hi @sidolov Sorry to ping you directly but today I see that you assisted me with all my PR. If you will have a time, please, check this one too.
Also, I want to excuse that didn't combine them into one but again this is a result of the fact that I have found all those issues separately. And when I made new PR other one was approved.
Thank you so much,
Alex

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants