Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Shipment] Add date when shipment has been shipped#11089

Merged
GSadee merged 2 commits into
Sylius:masterfrom
AdamKasp:add-shipped-at-date-to-shipment
Feb 11, 2020
Merged

[Shipment] Add date when shipment has been shipped#11089
GSadee merged 2 commits into
Sylius:masterfrom
AdamKasp:add-shipped-at-date-to-shipment

Conversation

@AdamKasp
Copy link
Copy Markdown
Contributor

@AdamKasp AdamKasp commented Feb 3, 2020

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

Screenshot 2020-02-11 at 09 27 00
Screenshot 2020-02-03 at 11 19 53

@AdamKasp AdamKasp requested a review from a team as a code owner February 3, 2020 10:20
Comment thread src/Sylius/Behat/Page/Admin/Order/ShowPage.php Outdated
Comment thread src/Sylius/Bundle/ShippingBundle/Resources/config/app/state_machine.yml Outdated
@AdamKasp AdamKasp force-pushed the add-shipped-at-date-to-shipment branch 2 times, most recently from b94c47f to 4728fd3 Compare February 3, 2020 11:33
Comment thread app/migrations/Version20200202104152.php Outdated
Comment thread src/Sylius/Behat/Context/Ui/Admin/ManagingShipmentsContext.php Outdated
@Zales0123 Zales0123 added the Feature New feature proposals. label Feb 3, 2020
Comment thread features/order/seeing_order_shipment_shipping_date.feature Outdated
Comment thread features/shipping/managing_shipments/managing_of_shipments_status.feature Outdated
Comment thread src/Sylius/Bundle/ShippingBundle/DateAssigner/ShipmentShippedAtDateAssigner.php Outdated
Comment thread src/Sylius/Bundle/ShippingBundle/DateAssigner/ShipmentShippedAtDateAssigner.php Outdated
Comment thread src/Sylius/Component/Shipping/Model/Shipment.php Outdated
Comment thread src/Sylius/Component/Shipping/Model/Shipment.php Outdated
Comment thread src/Sylius/Component/Shipping/Model/ShipmentInterface.php Outdated
@AdamKasp AdamKasp force-pushed the add-shipped-at-date-to-shipment branch 6 times, most recently from 6d2d890 to 285b228 Compare February 10, 2020 14:16
Comment thread psalm.xml
Comment thread src/Sylius/Behat/Page/Admin/Order/ShowPage.php Outdated
Comment thread src/Sylius/Bundle/AdminBundle/Resources/views/Order/Show/_shipment.html.twig Outdated
Comment thread src/Sylius/Bundle/ShippingBundle/spec/Provider/CalendarSpec.php Outdated
Comment thread src/Sylius/Bundle/ShippingBundle/spec/Provider/CalendarSpec.php Outdated
@AdamKasp AdamKasp force-pushed the add-shipped-at-date-to-shipment branch 2 times, most recently from b0ba60d to a044e55 Compare February 11, 2020 07:26
Comment thread src/Sylius/Behat/Resources/config/suites/ui/shipping/managing_shipments.yml Outdated
@AdamKasp AdamKasp force-pushed the add-shipped-at-date-to-shipment branch 2 times, most recently from f731212 to af64cf4 Compare February 11, 2020 08:10
@AdamKasp AdamKasp force-pushed the add-shipped-at-date-to-shipment branch from af64cf4 to 7aa394d Compare February 11, 2020 08:21
@GSadee GSadee merged commit 51cc069 into Sylius:master Feb 11, 2020
@GSadee
Copy link
Copy Markdown
Member

GSadee commented Feb 11, 2020

Thanks, Adam! 🥇

GSadee added a commit to Sylius/SyliusDemo that referenced this pull request Feb 11, 2020
This PR was merged into the 1.0-dev branch.

Discussion
----------

In Sylius we added new property on shipment entity - date when admin shipped a shipment (shippedAt)
here I added missing migration related with this PR Sylius/Sylius#11089

Commits
-------

db258a9 missing migration - shipped at date
billybrona pushed a commit to billybrona/SyliusDemo that referenced this pull request Nov 26, 2023
This PR was merged into the 1.0-dev branch.

Discussion
----------

In Sylius we added new property on shipment entity - date when admin shipped a shipment (shippedAt)
here I added missing migration related with this PR Sylius/Sylius#11089

Commits
-------

db258a9deb5a36743a7d894f189b1aa952904560 missing migration - shipped at date
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature New feature proposals.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants