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.

Sign up for GitHub

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

Jump to bottom

[Admin] Unify payment and shipment labels #10520

Merged
merged 1 commit into from Jul 11, 2019
Merged

[Admin] Unify payment and shipment labels #10520

merged 1 commit into from Jul 11, 2019

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented Jul 11, 2019

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

Payment index:
Before:
Zrzut ekranu 2019-07-11 o 08 28 49
After:
Zrzut ekranu 2019-07-11 o 08 25 58

Shipment index:
No changes:
Zrzut ekranu 2019-07-11 o 08 26 13

Order show:
Before:
Zrzut ekranu 2019-07-11 o 08 29 19
Zrzut ekranu 2019-07-11 o 08 29 10
After:
Zrzut ekranu 2019-07-11 o 08 26 38
Zrzut ekranu 2019-07-11 o 08 26 28

@GSadee GSadee added UX Issues and PRs aimed at improving User eXperience. Admin AdminBundle related issues and PRs. labels Jul 11, 2019
@GSadee GSadee requested a review from a team as a code owner July 11, 2019 06:35
@GSadee GSadee force-pushed the unify-payment-shipment-labels branch from 2a7e910 to 79f734a Compare July 11, 2019 07:03
@CoderMaggie
Copy link
Member

Thank you, Grzegorz! 🥇

@CoderMaggie CoderMaggie merged commit edf6def into Sylius:master Jul 11, 2019
@GSadee GSadee deleted the unify-payment-shipment-labels branch July 11, 2019 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tomanhez Tomanhez Tomanhez left review comments

@CoderMaggie CoderMaggie CoderMaggie approved these changes

@AdamKasp AdamKasp AdamKasp approved these changes

Assignees
No one assigned
Labels
Admin AdminBundle related issues and PRs. UX Issues and PRs aimed at improving User eXperience.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@GSadee @CoderMaggie @AdamKasp @Tomanhez