-
Notifications
You must be signed in to change notification settings - Fork 694
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
Replace RequestStore with ActiveSupport::CurrentAttributes #702
Replace RequestStore with ActiveSupport::CurrentAttributes #702
Conversation
Since I'm not entirely sure why the Fiber spec would pass on Rails 7.0 and not previous versions, but I'm not entirely sure we need that spec in this gem? ¯\_(ツ)_/¯ Even in current Rails, the default isolation level is set to |
👋 |
I can take a look at it... |
I can't remember offhand why we added the fiber spec. Probably sanity check for… something. 🤷 |
It was added by @the-spectator in his PR. |
Based on the work done here: #673 by @the-spectator