Skip to content
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

Adjusted currency symbol on bulk orders for issue #904 #1325

Closed
wants to merge 2 commits into from

Conversation

lin-d-hop
Copy link
Contributor

@lin-d-hop lin-d-hop commented Dec 9, 2016

Pulled in @Matt-Yorkleys changes from PR #1225 purely for the purposes of triggering a build for UK testing.

Addresses issue #904

Ready for testing by @OliverUK.

@lin-d-hop lin-d-hop added this to the UK Current milestone Dec 9, 2016
@lin-d-hop lin-d-hop removed this from the UK Current milestone Dec 9, 2016
@mkllnk mkllnk requested a deployment to staging December 9, 2016 11:56 Abandoned
@mkllnk mkllnk requested a deployment to staging December 9, 2016 12:13 Abandoned
@sstead
Copy link

sstead commented Dec 16, 2016

Looks good - the BOM Price column reflects the currency set in the instance's SuperAdmin Configuration
image
Have pushed to production.

@sstead sstead closed this Dec 16, 2016
@mkllnk mkllnk deleted the uk/currency_i18n branch December 16, 2016 03:48
@daniellemoorhead daniellemoorhead added this to the v1.8.7 - lucky seven milestone Dec 16, 2016
@daniellemoorhead
Copy link
Contributor

@lin-d-hop @Matt-Yorkley all done and merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants