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

Request: Conditional registration of webhooks for paid users only #703

Open
rkbhochalya opened this issue Mar 18, 2024 · 4 comments
Open
Labels

Comments

@rkbhochalya
Copy link

rkbhochalya commented Mar 18, 2024

Overview

As far as I know, there is currently no way to dynamically select the webhooks registered for a shop based on parameters such as subscription status.

We have an app in the Shopify App Store that monitors product updates. This app offers both free and paid plans. However, the free plan does not need monitoring product updates. While we check for active subscriptions while processing the webhooks and only proceed if the store has one, our servers still have to handle all those requests. We receive about 80% of requests that are unnecessary to process. These unnecessary requests use most of our server resources.

It would be great if we could register essential webhooks such as APP_UNINSTALLED for all stores, and some others like PRODUCTS_UPDATE for specific stores, like those on a paid plan. This would make webhook handling far more efficient for both our app and for Shopify.

@lizkenyon
Copy link
Contributor

Hi there 👋

Making it easier to subscribe to specific webhooks for certain shops easier is something we think would be a valuable add to the library.

Right now the workaround would be to manually make the API calls to create the webhook subscriptions for the shops that need the non-standard webhooks.

Our team is going to review this issue, and schedule it into our backlog. :)

@rkbhochalya
Copy link
Author

Hi Liz,

Thanks for getting back to me so quickly.

I have tried the workaround you suggested as well, but the problem with that is when calling the registerWebhooks function, it deletes any other webhooks that are not in the initial webhooks config.

The other alternative I think would be to register all webhooks using the REST or GraphQL API directly.

@lizkenyon
Copy link
Contributor

Hi! Yes, to clarify when I said manually I was referring to using the REST or GraphQL API directly.

Again, this is an area we are looking to improve, but for transparency this is not something will be completing in the immediate future.

Copy link
Contributor

We're labeling this issue as stale because there hasn't been any activity on it for 60 days. While the issue will stay open and we hope to resolve it, this helps us prioritize community requests.

You can add a comment to remove the label if it's still relevant, and we can re-evaluate it.

@github-actions github-actions bot added the Stale label May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants