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

Install the Checkout Glue API #2346

Merged
merged 8 commits into from
Jan 3, 2024
Merged

Install the Checkout Glue API #2346

merged 8 commits into from
Jan 3, 2024

Conversation

romanhavrylko
Copy link
Contributor

@romanhavrylko romanhavrylko added tech review needed PR needs tech review updated PR contains meaningful changes to some document labels Nov 24, 2023
DmitryLymarenko
DmitryLymarenko reviewed Nov 24, 2023
View reviewed changes
_includes/pbc/all/install-features/202403.0/install-glue-api/install-the-checkout-glue-api.md Outdated
Comment on lines 1 to 3



Copy link

Choose a reason for hiding this comment

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

do we need these spaces?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was moved from the previous version of this IG. @andriitserkovnyi, can we delete these empty lines?

Copy link
Collaborator

Choose a reason for hiding this comment

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

will do

_includes/pbc/all/install-features/202403.0/install-glue-api/install-the-checkout-glue-api.md Outdated Show resolved Hide resolved
@DmitryLymarenko DmitryLymarenko added PO review needed PR needs PO review and removed tech review needed PR needs tech review labels Nov 24, 2023
andriispryker
andriispryker previously approved these changes Nov 24, 2023
View reviewed changes
Copy link
Contributor

@andriispryker andriispryker left a comment

Choose a reason for hiding this comment

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

Approved.

@romanhavrylko romanhavrylko added TW review needed PR needs Technical writer's review and removed PO review needed PR needs PO review labels Nov 24, 2023
@andriitserkovnyi andriitserkovnyi self-assigned this Nov 27, 2023
@romanhavrylko romanhavrylko added the 202400 PR contains document for the upcoming release 202400 label Dec 19, 2023
@andriitserkovnyi andriitserkovnyi merged commit af39c1c into master Jan 3, 2024
12 of 13 checks passed
@andriitserkovnyi andriitserkovnyi deleted the bugfix/cc-31112-multi-call-of-cart-recalculation branch January 3, 2024 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andriispryker andriispryker andriispryker left review comments

@kkicheglovspryker kkicheglovspryker kkicheglovspryker left review comments

@DmitryLymarenko DmitryLymarenko DmitryLymarenko left review comments

@andriitserkovnyi andriitserkovnyi Awaiting requested review from andriitserkovnyi

@lenadoc lenadoc Awaiting requested review from lenadoc

Assignees

@andriitserkovnyi andriitserkovnyi

Labels
202400 PR contains document for the upcoming release 202400 TW review needed PR needs Technical writer's review updated PR contains meaningful changes to some document
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@romanhavrylko @DmitryLymarenko @andriitserkovnyi @kkicheglovspryker @andriispryker