Skip to content

Remove call to legacy_connection_handling #99

Merged
tvdeyen merged 1 commit intosolidusio:mainfrom
mamhoff:fix-rails-8-error
Feb 5, 2025
Merged

Remove call to legacy_connection_handling #99
tvdeyen merged 1 commit intosolidusio:mainfrom
mamhoff:fix-rails-8-error

Conversation

@mamhoff
Copy link
Contributor

@mamhoff mamhoff commented Feb 5, 2025

This setting doesn't actually do anything from Rails 7 onwards, and is
removed in Rails 7.1. Remove it.

This setting doesn't actually do anything from Rails 7 onwards, and is
removed in Rails 7.1. Remove it.
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks.

@mamhoff mamhoff changed the title Fix check for Rails version 7.0.x Remove call to legacy_connection_handling Feb 5, 2025
@tvdeyen tvdeyen enabled auto-merge February 5, 2025 12:58
@tvdeyen tvdeyen disabled auto-merge February 5, 2025 12:58
@tvdeyen tvdeyen merged commit 395d25e into solidusio:main Feb 5, 2025
3 checks passed
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