-
Notifications
You must be signed in to change notification settings - Fork 224
Unblock Partners API #6708
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
Unblock Partners API #6708
Conversation
Differences in type declarationsWe detected differences in the type declarations generated by Typescript for this branch compared to the baseline ('main' branch). Please, review them to ensure they are backward-compatible. Here are some important things to keep in mind:
New type declarationsWe found no new type declarations in this PR Existing type declarationspackages/cli-kit/dist/private/node/constants.d.ts@@ -29,7 +29,6 @@ export declare const environmentVariables: {
themeKitAccessDomain: string;
json: string;
neverUsePartnersApi: string;
- usePartnersApi: string;
skipNetworkLevelRetry: string;
maxRequestTimeForNetworkCalls: string;
};
|
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success3487 tests passing in 1406 suites. Report generated by 🧪jest coverage report action from 3495253 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks reasonable, works as described, solves the problem. Unfortunate this is breaking things for users, as we do want to fully drop Partners API support!
dmerand
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tophat LGTM
Merge activity
|
WHY are these changes introduced?
Fixes https://github.com/Shopify/shopify-dev/issues/65805
Related to https://community.shopify.dev/t/unable-to-migrate-extensions-and-deploy/26901
In #6645 we blocked Partners API by default, but it's still needed to import extensions.
Discussion about the approach: https://shopify.slack.com/archives/C07UJ7UNMTK/p1765301233241769
WHAT is this pull request doing?
Basically reverts:
How to test your changes?
p shopify app info --reset --verbose: uses both APIsSHOPIFY_CLI_NEVER_USE_PARTNERS_API=1 p shopify app info --reset --verbose: does not use Partners APIMeasuring impact
How do we know this change was effective? Please choose one:
Checklist