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

new sdk 2.8.7 changes #503

Merged
merged 2 commits into from
Sep 11, 2023
Merged

new sdk 2.8.7 changes #503

merged 2 commits into from
Sep 11, 2023

Conversation

poojarazor
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....

Copy link

@ankitdas13 ankitdas13 left a comment

Choose a reason for hiding this comment

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

Before merge, please check other api's (if woocommerce is using other api rather than order create) as well, because now in sdk we are handling v1 & v2 api version both.

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

@poojarazor poojarazor added the TestingNotRequired TestingNotRequired label for BVT label Sep 11, 2023
@poojarazor poojarazor merged commit ffcc9fb into master Sep 11, 2023
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.

3 participants