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

[Shipment] Ship a shipment with tracking code on the list #11066

Merged

Conversation

AdamKasp
Copy link
Contributor

@AdamKasp AdamKasp commented Jan 26, 2020

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

Now Administrator can process shipment directly from the index page by providing a tracking code with ship action, i also deleted the customer column because this information is unimportant in in Shipping process, ofc, if Admin will need this information, by order number, can check all data about the shipment.

Screenshot 2020-01-26 at 18 12 30

@AdamKasp AdamKasp requested a review from a team as a code owner January 26, 2020 17:17
@AdamKasp AdamKasp force-pushed the providing-tracking-code-on-shipment-index branch from fed1545 to 9519c18 Compare January 28, 2020 18:25
@AdamKasp AdamKasp changed the title [WIP][Shipment] ship and providing tracking code on shipment index [Shipment] ship and providing tracking code on shipment index Jan 28, 2020
@GSadee GSadee added Admin AdminBundle related issues and PRs. Feature New feature proposals. labels Jan 29, 2020
@AdamKasp AdamKasp changed the title [Shipment] ship and providing tracking code on shipment index [Shipment] Ship a shipment with tracking code on the list Jan 29, 2020
@AdamKasp AdamKasp force-pushed the providing-tracking-code-on-shipment-index branch 4 times, most recently from d00daa0 to 250badd Compare January 30, 2020 12:14
@AdamKasp AdamKasp force-pushed the providing-tracking-code-on-shipment-index branch 4 times, most recently from 140ebc0 to 627f1ce Compare January 31, 2020 09:08
@AdamKasp AdamKasp force-pushed the providing-tracking-code-on-shipment-index branch from 627f1ce to d2d7de9 Compare January 31, 2020 13:04
@GSadee GSadee force-pushed the providing-tracking-code-on-shipment-index branch from d2d7de9 to 4318e92 Compare February 3, 2020 07:45
@GSadee GSadee force-pushed the providing-tracking-code-on-shipment-index branch from 84a1078 to b0d5fce Compare February 3, 2020 10:06
@lchrusciel lchrusciel merged commit a82e52d into Sylius:master Feb 3, 2020
@lchrusciel
Copy link
Member

Thanks, Adam! 🎉

GSadee added a commit that referenced this pull request Feb 5, 2020
This PR was merged into the 1.7-dev branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | master
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | fixes for #11066 (comment)
| License         | MIT

Based on #11066

<!--
 - Bug fixes must be submitted against the 1.6 branch (the lowest possible)
 - Features and deprecations must be submitted against the master branch
 - Make sure that the correct base branch is set
-->


Commits
-------

782cc2e [Admin] Use shipment action for shipment grid
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin AdminBundle related issues and PRs. Feature New feature proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants