-
Notifications
You must be signed in to change notification settings - Fork 58
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
Banner: API Rate Limit #551
Comments
@ryanwaits Let's split this issue into 2 :
|
@saralab Please refer only to the board shared for this specific task (https://www.figma.com/file/Gc7LJh8NWRvSOWQMnxmCm5/Hiro-Docs-2023?type=design&node-id=621%3A128&mode=design&t=zuxgF0HQ6JBj9UaB-1) The other Figma board (warning banner) was a previous design made as an example, not meant to be implemented right now. That being said, if there's capacity to implement this as well, that's good! |
based on how the current banner works, we should be able to put up a new one once the blog is posted. it's a hard coded time window so if we have an expected release date of the the rate limit blog + how long we'd like to have the banner exposed, all i'd need are the dates and the expected url for the blog and we should be good to go |
|
Aside from the Banner or however else y'all are thinking about solving this, please also ensure all the three rate-limiting guides are updated to include the supported RPM with and without API Key. |
I understand that, the ask to create an issue is so we can track that change even if we don't have the bandwidth at the moment. |
as part of comms for a new API rate limit, I'd like to put a banner up on docs. Ideally the banner goes up once the blog is out, but it can go up anytime if there are other pressing priorities.
@Ginny /-/ has this banner preset for announcements/warning, we'd only need to swap colors and icon
Copy: Upcoming changes to rate limits for Hiro APIs → Learn more.
https://www.figma.com/file/Gc7LJh8NWRvSOWQMnxmCm5/Hiro-Docs-2023?type=design&node-id=621%3A128&mode=design&t=zuxgF0HQ6JBj9UaB-1
@sabbyanandan @eugeniadigon
The text was updated successfully, but these errors were encountered: