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

Leftover deploy pt. 2 - March 1, 2024 #6988

Merged
merged 17 commits into from Mar 1, 2024
Merged

Leftover deploy pt. 2 - March 1, 2024 #6988

merged 17 commits into from Mar 1, 2024

Conversation

internetisaiah
Copy link
Contributor

@internetisaiah internetisaiah commented Mar 1, 2024

#6970 - Update talonone.md
#6977 - add cloudflare ip ranges
#6978 - CO-2350 Add catalog permissions.
#6972 - Troubleshooting push not sending - Android
#6980 - Update index.md
#6981 - Update ab_test_projection.md
#6983 - Add Show archived content

sambellbraze and others added 17 commits February 28, 2024 17:12
I was working with a customer who has Talon.One and they said the T1 docs said a 2,500 rate limit is what T1 stipulate. Whereas the Braze docs says 5,000.

T1 docs are here: https://docs.talon.one/docs/dev/technology-partners/braze#:~:text=Braze%20supports%20per%2Dminute%20rate%20limiting%20for%20sending%20messages.%20To%20ensure%20that%20it%20matches%20the%20default%20per%2Dsecond%20rate%20limit%20of%20our%20CEP%20endpoints%2C%20in%20your%20Braze%20settings%2C%20set%20Delivery%20Speed%20Rate%2DLimiting%20to%20a%20maximum%20of%202500%20requests%20per%20minute%20or%20lower%2C%20if%20required.

I reached our via our Partnerships team and T1 came back and said:

"Talon.One can easily handle 300 requests per second (hence 18 000 requests per minute) 
BUT they need to be evenly distributed (which is the issue as Braze does not distribute evenly)
For the moment 2500 requests per minute seems to be the recommended limit to make sure the bursts don't impact the client while the teams are working towards increasing the limits for large clients such as Costa Coffee or Flink"
BD-2872 Troubleshooting push not sending - Android
BD-2875 Add Show archived content
@cla-bot cla-bot bot added the cla-signed label Mar 1, 2024
@internetisaiah internetisaiah marked this pull request as ready for review March 1, 2024 22:17
@internetisaiah internetisaiah changed the title Ad hoc deploy Ad hoc deploy - March 1, 2024 Mar 1, 2024
@lydia-xie lydia-xie changed the title Ad hoc deploy - March 1, 2024 Leftover deploy pt. 2 - March 1, 2024 Mar 1, 2024
Copy link
Contributor

@rachel-feinberg rachel-feinberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rachel-feinberg rachel-feinberg merged commit 8950ae0 into master Mar 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants