-
Notifications
You must be signed in to change notification settings - Fork 125
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
Remove API Client config from link
command
#3411
Conversation
Thanks for your contribution! Depending on what you are working on, you may want to request a review from a Shopify team:
|
Coverage report
Show files with reduced coverage 🔻
Test suite run success1580 tests passing in 738 suites. Report generated by 🧪jest coverage report action from 8e37004 |
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.
I've tried creating a new app and running link, but I'm getting an error:
Fix a schema error in /Users/gonzalo/tests/2024-02-09/shopify.app.toml: │
│ [ │
│ { │
│ "code": "invalid_type", │
│ "expected": "string", │
│ "received": "undefined", │
│ "path": [ │
│ "name" │
│ ], │
│ "message": "Required" │
│ }, │
│ { │
│ "code": "invalid_type", │
│ "expected": "object", │
│ "received": "undefined", │
│ "path": [ │
│ "auth" │
│ ], │
│ "message": "Required" │
│ }, │
│ { │
│ "code": "invalid_type", │
│ "expected": "object", │
│ "received": "undefined", │
│ "path": [ │
│ "webhooks" │
│ ], │
│ "message": "Required" │
│ }, │
│ { │
│ "code": "invalid_type", │
│ "expected": "string", │
│ "received": "undefined", │
│ "path": [ │
│ "application_url" │
│ ], │
│ "message": "Required" │
│ }, │
│ { │
│ "code": "invalid_type", │
│ "expected": "boolean", │
│ "received": "undefined", │
│ "path": [ │
│ "embedded" │
│ ], │
│ "message": "Required" │
│ } │
│ ]
'@shopify/app': patch | ||
--- | ||
|
||
Remove api client configuration from the link command |
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.
This is transparent for the users, I think we don't need a changelog entry.
3564dab
to
8e37004
Compare
WHY are these changes introduced?
Partially Fixes: https://github.com/Shopify/develop-app-management/issues/1612
Versioned app configuration has been 100% rolled out altogether with the 3.55.0 release. That means that the CLI should not use
api client configuration
anymoreWHAT is this pull request doing?
link
command is not merging the remote API Client configurationAPI Client configuration
from theapp
API requestsHow to test your changes?
link
command should work as expectedPost-release steps
Measuring impact
How do we know this change was effective? Please choose one:
Checklist
dev
ordeploy
have been reflected in the internal flowchart.