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

[Api][Test] Add default currency #5607

Merged

Conversation

Arminek
Copy link
Contributor

@Arminek Arminek commented Jul 26, 2016

Q A
Bug fix? yes
New feature? no
BC breaks? no
Related tickets #5592
License MIT

@@ -4,6 +4,13 @@ Sylius\Component\Core\Model\Channel:
name: "Web Channel"
hostname: "localhost"
description: "Lorem ipsum"
defaultCurrency: '@currency'
Copy link
Contributor

@pamil pamil Jul 26, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"@currency"?

@Arminek Arminek force-pushed the quick-fix-default-currency branch from 6ebad7e to 91b5f06 Compare July 26, 2016 12:17
@Arminek Arminek force-pushed the quick-fix-default-currency branch from 91b5f06 to be89ee7 Compare July 26, 2016 12:19
@michalmarcinkowski michalmarcinkowski merged commit f65866a into Sylius:master Jul 26, 2016
@michalmarcinkowski
Copy link
Contributor

Thanks Arek! 👍

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

3 participants