Skip to content

Conversation

@camallen
Copy link

@camallen camallen commented Nov 14, 2025

Add rails 8 rate_limit activation error

If you are willing to accept this PR i'll tidy it up and add tests


Please keep these instructions in mind so we can review it more efficiently:

  • Add the references of all the related GH issues, Linear IDs or PRs in the description
  • Whether it's a new feature or a bug fix, make sure they're covered by new test cases
  • If this PR contains any refactoring work, please give it its own commit(s)
  • Finally, please add an entry to the corresponding changelog
  • PR title should use conventional commit style (feat:, fix:, ref:, meta:)

Other Notes

Description

Describe your changes:

Copy link
Collaborator

@solnic solnic left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Please add tests too and we could merge this in 😄

"ActionController::NotImplemented",
"ActionController::ParameterMissing",
"ActionController::RoutingError",
"ActionController::TooManyRequests",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add it conditionally if it's rails >= 8.0

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