Skip to content

THREESCALE-12434: Migrate from protected attributes to strong parameters - Part 3#4255

Open
mayorova wants to merge 4 commits intostrong-params-part2from
strong-params-part3
Open

THREESCALE-12434: Migrate from protected attributes to strong parameters - Part 3#4255
mayorova wants to merge 4 commits intostrong-params-part2from
strong-params-part3

Conversation

@mayorova
Copy link
Contributor

What this PR does / why we need it:

This is part 3 (final) of the migration from protected attributes to strong parameters. See also:

Protected attributes is an old Rails feature which was deprecated a long time ago. We were using protected_attributes_continued gem to keep it working, but now it's also discontinued and does not support Rails 7+, so it's a blocker for upgrading to Rails 7.2 for us.

This part clears up all the remaining references to protected attributes, and removes the protected_attributes_continued gem.

Which issue(s) this PR fixes

https://redhat.atlassian.net/browse/THREESCALE-12434

Verification steps

All tests should pass, and all features should work as before.

Special notes for your reviewer:

@qltysh
Copy link

qltysh bot commented Mar 19, 2026

❌ 7 blocking issues (7 total)

Tool Category Rule Count
rubocop Lint Class has too many lines. [206/200] 5
rubocop Lint Block has too many lines. [118/25] 1
rubocop Lint Assignment Branch Condition size for create\_a\_complete\_provider is too high. [<16, 154, 1> 154.8/20] 1

@mayorova mayorova changed the title Strong params part3 THREESCALE-12434: Migrate from protected attributes to strong parameters - Part 3 Mar 19, 2026
@mayorova mayorova force-pushed the strong-params-part3 branch from 3464b1d to c9f7b0d Compare March 19, 2026 18:21
@mayorova mayorova force-pushed the strong-params-part3 branch from c9f7b0d to a37003a Compare March 20, 2026 12:05
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.

1 participant