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

Fulfillment app docs #2251

Merged
merged 46 commits into from
Nov 9, 2023
Merged

Fulfillment app docs #2251

merged 46 commits into from
Nov 9, 2023

Conversation

andriitserkovnyi
Copy link
Collaborator

PR Description

Fulfillment app docs

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md


Once the picking process is complete the state in the State Machine is updated to Picking Finished. To set up State Machine configuration, go (here)[https://docs.spryker.com/docs/scos/dev/feature-integration-guides/{{page.version}}/install-the-push-notification-feature.html#set-up-configuration].

A subprocess for DummyPayment01 below describes in detail the state transition for the sales order line item:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Do we even need to describe the following?

Copy link
Contributor

Choose a reason for hiding this comment

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

We can mention at least that its integrated into the state machine as a subprocess in the example DummyPayment01 (for B2C demoshop) and also MarketplacePayment01 (for B2C Marketplace demoshop). For example we describe some already here. https://docs.spryker.com/docs/pbc/all/order-management-system/202307.0/base-shop/manage-in-the-back-office/orders/change-the-state-of-order-items.html#reference-information-changing-the-state-of-order-items

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Does it make sense to update change the state of order items with the statuses from the fulfillment app state machine?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can add those there, yes.

Copy link
Contributor

Choose a reason for hiding this comment

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

It was flagged by QA, please ensure that changes in the MarketplacePayment01 also described in docs.


Once the picking process is complete the state in the State Machine is updated to Picking Finished. To set up State Machine configuration, go (here)[https://docs.spryker.com/docs/scos/dev/feature-integration-guides/{{page.version}}/install-the-push-notification-feature.html#set-up-configuration].

A subprocess for DummyPayment01 below describes in detail the state transition for the sales order line item:
Copy link
Contributor

Choose a reason for hiding this comment

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

We can mention at least that its integrated into the state machine as a subprocess in the example DummyPayment01 (for B2C demoshop) and also MarketplacePayment01 (for B2C Marketplace demoshop). For example we describe some already here. https://docs.spryker.com/docs/pbc/all/order-management-system/202307.0/base-shop/manage-in-the-back-office/orders/change-the-state-of-order-items.html#reference-information-changing-the-state-of-order-items


Once the picking process is complete the state in the State Machine is updated to Picking Finished. To set up State Machine configuration, go (here)[https://docs.spryker.com/docs/scos/dev/feature-integration-guides/{{page.version}}/install-the-push-notification-feature.html#set-up-configuration].

A subprocess for DummyPayment01 below describes in detail the state transition for the sales order line item:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can add those there, yes.

andriispryker
andriispryker previously approved these changes Nov 3, 2023
andriispryker
andriispryker previously approved these changes Nov 6, 2023
@andriitserkovnyi andriitserkovnyi self-assigned this Nov 8, 2023
@andriitserkovnyi andriitserkovnyi added the new PR contains a new document label Nov 8, 2023
@andriitserkovnyi andriitserkovnyi merged commit 8abd6aa into master Nov 9, 2023
13 checks passed
@andriitserkovnyi andriitserkovnyi deleted the fulfillment-app branch November 9, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new PR contains a new document
Development

Successfully merging this pull request may close these issues.

2 participants