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

remove <= 7.1 rails requirement #98

Closed
wants to merge 1 commit into from

Conversation

jasonayre
Copy link

@jasonayre jasonayre commented May 21, 2024

Unless I am missing something, I didn't see a reason for the < 7.1 version specificity, and it was preventing me from using (my trax_model gem depends on it) in a new rails 7.1.3 project

@jrafanie
Copy link
Collaborator

This is the same as #96 and similar to #92. As a contributor to this repo in the past, I'm hoping to be given permission to take over maintenance of this project so I can release it with support for rails 7.0 and 7.1. 🤞

That would be favorable to releasing a fork of it.

For now, you'll need to use your fork with #96 applied.

@FooBarWidget, thanks for the work on this gem and please reach out to me if this is possible.

@EOengineer
Copy link

Just checking to see if there has been any movement on this. Currently attempting a Rails 7.1.3 upgrade and default_values_for is a blocker.

@jrafanie
Copy link
Collaborator

jrafanie commented Jul 10, 2024

@EOengineer I've reached out to many prior committers and a few have vouched for me being added but they don't have the owner permission to add me. I'm still hoping to get a few minutes of @FooBarWidget's widget time so I can be added. The alternative is to fork the gem and run from that fork's branch/tag or fork it as a new gem and release it.

@FooBarWidget
Copy link
Owner

Hello sorry for the late reply. I've now assigned @jrafanie as a maintainer.

@jrafanie
Copy link
Collaborator

I've merged #96 and pushed 3.5.0 with rails 7.0 support and 3.6.0 with rails 7.1 support. Please open issues if you find any problems. Thanks!

@jrafanie jrafanie closed this Jul 16, 2024
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.

4 participants