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

mws-3953 -- bulk api with subscribers and topic details #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shiva-sc
Copy link
Contributor

@shiva-sc shiva-sc commented Jan 9, 2025

No description provided.

@jimleroyer
Copy link

jimleroyer commented Jan 13, 2025

Hi Shiva! Jimmy from the GCNotify team. The code will work to sent the bulk to send to GCNotify from a code reading (we have not tested it). The one note I would leave is to make sure to have proper error handling when interacting with the API, i.e. if you get a http 500 class failure, you can try again with an exponential backoff retry strategy. You could have a sophisticated mechanism with queues or queue these async for later with a timer (up for a max number of times). If you don't try again on failures, you might have to manage some of these manually otherwise.

Hope that helps!

@duboisp duboisp marked this pull request as draft January 15, 2025 15:56
@shiva-sc shiva-sc force-pushed the bulk-api-demo branch 9 times, most recently from 475eb42 to 80b8ef0 Compare February 7, 2025 15:53
@shiva-sc shiva-sc marked this pull request as ready for review February 12, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants