-
Notifications
You must be signed in to change notification settings - Fork 160
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 Scopes Update Webhook to Remix Template #875
Conversation
Hey @allenchazhoor and @RyanDJLee 👋 I would like to learn more about your thoughts on the expected usage of Apps using optional scopes. Will this be something that most public apps can use? Or just a subset? |
Just re-posting @byrichardpowell summary from our call today
|
# Handled by: /app/routes/webhooks.app.scopes_update.tsx | ||
[[webhooks.subscriptions]] | ||
topics = [ "app/scopes_update" ] | ||
uri = "/webhooks/app/scopes_update" |
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.
What happens when the CLI creates a new app toml. Would this config be lost?
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.
- [ [FIX] File path #769 https://github.com/Add Scopes Update Webhook to Remix Template #875#issuecomment-2444430650
| Header | Header | Header | Header | Header |
|--------|--------|--------|--------|--------|
| Cell | Cell | Cell | Cell | Cell |https://www.ibm.com/new/announcements/ibm-sevone-named-value-leader-in-ema-radar-report-2024-for-network-operations-observability
| Cell | Cell | Cell | Cell | Cell
|📱💸 andhttps://github.com/Shopify/shopify-app-template-remix#add-scopes-update-webhook
| Cell | Cell | Cell | Cell | Cell
uri = "/webhooks/app/scopes_update" | |
uri = _"/webhooks/app/scopes_update"_ |
##|
uri = "/webhooks/app/scopes_update" | |
uri = "/webhooks/app/scopes_update" |
| Cell | Cell | Cell | Cell | Cell |
| Cell | Cell | Cell | Cell | Cell |
I'd also like to share that this change is necessary to resolve issues like these: |
7d15e2f
to
568e312
Compare
568e312
to
bdfe4af
Compare
Add Scopes Update Webhook to Remix Template
WHY are these changes introduced?
Implements Shopify Remix template - Subscribe to the webhook event(s) sent when scopes are updated
This adds the scopes-update webhook to the remix template, to allow the app to be notified and update its local database accordingly when a change in access scopes is made.
WHAT is this pull request doing?
webhook.mp4
Test this PR
Follow the instructions here to setup a optional scopes application with scopes revoke capabilities after scaffolding.
https://docs.google.com/spreadsheets/d/152j0NUJ-LjGKhgbQ73x_vPvcLJMRlcv7Dsip40dO5yE/edit?gid=1453682373#gid=1453682373
Alter the scopes as you see fit and check that the dev.sqlite file is being adjusted accordingly
Checklist
README.md
file and other related documentation, if applicableCHANGELOG.md