-
Notifications
You must be signed in to change notification settings - Fork 685
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
Add host to query param and consume App Bridge v2 #1376
Conversation
29cd7d4
to
e70c291
Compare
Given the issue we are experiencing with the |
Tested this PR with and without host param ✅ Without host param, expect to see error ✅ With host param, expect to install the app normally |
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.
Thank you for the fix...! 🎩 w/ Henry demoing the changes 👍
If it's tophatted, let's ship it! 🚀 😄 |
What this PR does
Resolves https://github.com/Shopify/app-bridge/issues/2195
Reviewer's guide to testing
Things to focus on
Test environment:
3.0.3p157
6.0.4.6
2.3.5
v17.4.0
gem 'shopify_app', path: '~/src/github.com/Shopify/shopify_app'
yarn add @rails/webpacker
. For some reason, my environment doesn't haverails/webpacker
by default./bin/webpack
shopify app serve
, this will also output a ngrok urlapp/assets/javascripts/shopify_app/redirect.js
to see more output in consoleTest new installation flow
Test the login page on shop already has the app installed
Test re-authentication. Ex: cookies is expired
db/development.sqlite3
with https://kepler.shopifycloud.com/services/db-browser-for-sqliteTest AppStore installation flow
Add app
from the AppStore will redirect to partner application url with hmac, host, shop, and timestamp in the query params.Test ITP
Note: if you click
Don't allow
in the ITP screen, you will be redirected to the apps list. The ITP screen will show up again every time you open embedded apps.Checklist
Before submitting the PR, please consider if any of the following are needed:
CHANGELOG.md
if the changes would impact usersREADME.md
, if appropriate./docs
, if necessary