Skip to content

style: add rate limit details#13104

Merged
rjan90 merged 1 commit intofilecoin-project:masterfrom
barbaraperic:bp/too-many-requests-message
May 7, 2025
Merged

style: add rate limit details#13104
rjan90 merged 1 commit intofilecoin-project:masterfrom
barbaraperic:bp/too-many-requests-message

Conversation

@barbaraperic
Copy link
Contributor

@barbaraperic barbaraperic commented May 7, 2025

Related Issues

Issue #13063
This PR adds rate limit details to "Too many requests: Wallet limit" error message because it lacks clarity. See UXIT dogfooding report.

Proposed Changes

Added rate limit details - "You can request tFIL every 2 hours".

Additional Info

Screenshot 2025-05-07 at 12 49 13

Checklist

Before you mark the PR ready for review, please make sure that:

@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz May 7, 2025
@rjan90 rjan90 added the skip/changelog This change does not require CHANGELOG.md update label May 7, 2025
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting Review in FilOz May 7, 2025
Copy link
Contributor

@rjan90 rjan90 left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from 🔎 Awaiting Review to ✔️ Approved by reviewer in FilOz May 7, 2025
@rjan90 rjan90 merged commit 21392a5 into filecoin-project:master May 7, 2025
97 of 99 checks passed
@github-project-automation github-project-automation bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz May 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants