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

[Order] Change Adjustment description to label #3880

Merged

Conversation

Zales0123
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Fixed tickets #3849
License MIT
Doc PR Sylius/Sylius-Docs#396

@pjedrzejewski pjedrzejewski added the DX Issues and PRs aimed at improving Developer eXperience. label Jan 14, 2016
@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 05e4393 to 5cadc89 Compare January 14, 2016 12:45

$order->addAdjustment($adjustment);
}


Copy link
Contributor

Choose a reason for hiding this comment

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

Should be reverted.

@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 5cadc89 to 333ecd0 Compare January 14, 2016 12:59
@michalmarcinkowski michalmarcinkowski added BC Break PRs introducing BC breaks (do not even try to merge). Ready labels Jan 14, 2016
@peteward
Copy link

👍

1 similar comment
@mckelviemark
Copy link

👍

@michalmarcinkowski michalmarcinkowski added this to the v0.17.0 milestone Jan 18, 2016
@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 333ecd0 to e63124a Compare January 20, 2016 14:10
@Zales0123
Copy link
Member Author

@pjedrzejewski @michalmarcinkowski same here, Scrutinizer error :(

@pjedrzejewski
Copy link
Member

@Zales0123 Please update UPGRADE file.

@Zales0123
Copy link
Member Author

@pjedrzejewski done 👍

@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 890bce5 to acb3f60 Compare January 26, 2016 09:51
@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 8b8028e to 1fb0643 Compare January 26, 2016 12:26
{
/** @var $promotion PromotionInterface */
$promotion = $this->getPromotionFactory()->createNew();
$promotion->setName($name);
$promotion->setDescription($description);
$promotion->setDescription($label);
Copy link
Contributor

Choose a reason for hiding this comment

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

setDescription($label)?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point 😄

@Zales0123 Zales0123 force-pushed the adjustment-description-to-label branch from 1fb0643 to 0ec405d Compare January 26, 2016 13:11
pjedrzejewski pushed a commit that referenced this pull request Jan 26, 2016
[Order] Change Adjustment description to label
@pjedrzejewski pjedrzejewski merged commit 3723d36 into Sylius:master Jan 26, 2016
@pjedrzejewski
Copy link
Member

Thanks Mateusz!

@Zales0123 Zales0123 deleted the adjustment-description-to-label branch March 23, 2016 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BC Break PRs introducing BC breaks (do not even try to merge). DX Issues and PRs aimed at improving Developer eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants