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

DEV: Fix reading_role/writing_role deprecations #48

Merged
merged 6 commits into from
Apr 6, 2023

Conversation

CvX
Copy link
Contributor

@CvX CvX commented Jun 20, 2022

Introduced in rails/rails#42445

Example of a warning:

DEPRECATION WARNING: ActiveRecord::Base.writing_role is deprecated and will be removed in Rails 7.1.
Use `ActiveRecord.writing_role` instead.
 (called from block (2 levels) in <top (required)> at /__w/discourse/discourse/spec/rails_helper.rb:304)

Introduced in rails/rails#42445

Example of a warning:
```
DEPRECATION WARNING: ActiveRecord::Base.writing_role is deprecated and will be removed in Rails 7.1.
Use `ActiveRecord.writing_role` instead.
 (called from block (2 levels) in <top (required)> at /__w/discourse/discourse/spec/rails_helper.rb:304)
```
@CvX CvX marked this pull request as ready for review June 21, 2022 22:15
@CvX CvX requested a review from tgxworld June 23, 2022 11:24
Copy link
Contributor

@tgxworld tgxworld left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this @CvX 👍 Do remember to update the CHANGELOG before merging.

@tgxworld
Copy link
Contributor

Ping @CvX

@CvX CvX force-pushed the ar-deprecations branch 23 times, most recently from e0ac6da to b6eb199 Compare April 5, 2023 17:01
@CvX CvX force-pushed the ar-deprecations branch 2 times, most recently from 485b8f6 to 66e27bc Compare April 5, 2023 17:18
@oblakeerickson oblakeerickson merged commit cfcf68a into main Apr 6, 2023
@oblakeerickson oblakeerickson deleted the ar-deprecations branch April 6, 2023 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants