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

6.1 configurations has symbol keys, [] deprecated, use configs_for #189

Merged
merged 1 commit into from
Sep 1, 2022

Conversation

jrafanie
Copy link
Member

@jrafanie jrafanie commented Sep 1, 2022

Fixes ManageIQ/manageiq-appliance#366

DEPRECATION WARNING: [] is deprecated and will be removed from Rails 6.2 (Use configs_for)

This fixes the deprecation
This now uses symbols for configuration keys (since rails 6.1 has changed this as well)

Similar to: ManageIQ/manageiq@0d57f5e

Fixes ManageIQ/manageiq-appliance#366

DEPRECATION WARNING: [] is deprecated and will be removed from Rails 6.2 (Use configs_for)

This fixes the deprecation
This now uses symbols for configuration keys (since rails 6.1 has changed this as well)

Similar to: ManageIQ/manageiq@0d57f5e
@jrafanie
Copy link
Member Author

jrafanie commented Sep 1, 2022

cc @kbrock

@kbrock kbrock self-assigned this Sep 1, 2022
@kbrock kbrock mentioned this pull request Sep 1, 2022
14 tasks
@kbrock kbrock merged commit 6304e53 into ManageIQ:master Sep 1, 2022
@jrafanie jrafanie deleted the fix_rails61_db_connections_check branch September 1, 2022 19:43
@jrafanie
Copy link
Member Author

jrafanie commented Sep 1, 2022

I'm guessing we'll need a new release of this so we can use it going forward

Fryguy added a commit that referenced this pull request Sep 2, 2022
Fixes:
- 6.1 configurations has symbol keys, [] deprecated, use configs_for [[#189](#189)]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal DB create fails due to existing connections
2 participants