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

[PO-212]:-Enqueue checkout.js to checkout page #573

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

yashgit891
Copy link
Contributor

Note :- Please follow the below points while attaching test cases document link below:

- If label Tested is added then test cases document URL is mandatory.

- Link added should be a valid URL and accessible throughout the org.

- If the branch name contains hotfix / revert by default the BVT workflow check will pass.

Test Case Document URL
Please paste test case document link here....

@yashgit891 yashgit891 changed the title Enqueue checkout.js to checkout page [PO-212]:-Enqueue checkout.js to checkout page Sep 20, 2024
@yashgit891 yashgit891 added the TestingNotRequired TestingNotRequired label for BVT label Sep 20, 2024
Copy link
Contributor

@abdulwahidsharief abdulwahidsharief left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@abdulwahidsharief abdulwahidsharief left a comment

Choose a reason for hiding this comment

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

LGTM

@yashgit891 yashgit891 marked this pull request as ready for review September 20, 2024 11:51
@yashgit891 yashgit891 merged commit a83468d into master Sep 20, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TestingNotRequired TestingNotRequired label for BVT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants