Skip to content

Support for Rails 7.2 #2411

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

Open
dwarner opened this issue Oct 4, 2024 · 19 comments
Open

Support for Rails 7.2 #2411

dwarner opened this issue Oct 4, 2024 · 19 comments

Comments

@dwarner
Copy link

dwarner commented Oct 4, 2024

I know 7.1.0 just got released with support for Rails 7.1.x (thanks), but is there an ETA for Rails 7.2 support?

https://github.com/rsim/oracle-enhanced/blob/v7.1.0/activerecord-oracle_enhanced-adapter.gemspec
s.add_runtime_dependency("activerecord", ["~> 7.1.0"])

I know Rails 7.2 ActiveRecord has a lot of "Remove deprecated" . . .
https://guides.rubyonrails.org/7_2_release_notes.html#active-record

@numbcoder
Copy link

+1

1 similar comment
@matthewkrom
Copy link

+1

@rjschof
Copy link

rjschof commented Jan 17, 2025

+1. Any updates on this or any sort of plans to maintain this gem?

@andynu
Copy link
Contributor

andynu commented Jan 22, 2025

I started attempting to upgrade to 7.2, and got stuck. #2422
Anyone willing to give my very limited progress a look, I'd appreciate additional perspectives and/or suggestions for diagnosing.

@andynu
Copy link
Contributor

andynu commented Jan 31, 2025

I've made further progress and cut a new PR for consideration. Not all the way there, but a significant step closer. There are now only four remaining test failures locally. #2424

@dwarner
Copy link
Author

dwarner commented May 15, 2025

@andynu Thanks for your hard work!

We've pulled your 7.2 branch into one of our projects and have NOT run into a problem.

We'll be trying it on other projects coming up.

Do you have an ETA on an official release?

@andynu
Copy link
Contributor

andynu commented May 15, 2025 via email

@akostadinov
Copy link
Contributor

Whoever tested this, maybe you add an approval so hopefully author attention is captured.

@joshRpowell
Copy link

joshRpowell commented May 15, 2025

@andynu Thanks for your hard work!

We've pulled your 7.2 branch into one of our projects and have NOT run into a problem.

We'll be trying it on other projects coming up.

Do you have an ETA on an official release?

+1 branch has been running in app ~6mo now. No issues.
Thanks for all your work @andynu 🥳
(unfortunately, not able to approve)

@dwarner
Copy link
Author

dwarner commented May 15, 2025

@andynu I will make a comment on the PR.

Have you gotten any responses/feedback from the maintainers?

@andynu
Copy link
Contributor

andynu commented May 15, 2025 via email

@dwarner
Copy link
Author

dwarner commented May 15, 2025

@rsim Are you still maintaining this project?

@smarks-1-ND
Copy link

Yes, thank you @andynu . I am testing your branch now and things are looking good. Hoping this can be merged soon.

@Bjoernsen
Copy link

Thank you @andynu ! I have a quite complex gem that connects to different databases.

Most of my tests worked out of the box with 7.2. Only one test failed. I am not sure if this is a Rails 7.2 behavior, a bug in the adapter, or if I do not know what I am doing.

In my gem, I have a method calling something like

SomeValue.where('value BETWEEN ? and ?', min, max)

The values for min and max come from another DB call. In 7.1 it works, in 7.2 I get the error ActiveRecord::StatementInvalid: OCIError: ORA-01722: invalid number

The issue is that min and max can be a BigDecimal (e.g. 0.23255e3).

The solution was to add .to_f to the variables.

SomeValue.where('value BETWEEN ? and ?', min.to_f, max.to_f)

@net1957
Copy link

net1957 commented May 16, 2025

Hello,
I tried @andynu fork on my little application connected to an old Oracle 11 DB.
It worked without any problem after the update of the application to Rails 7.2.2.1.

Hope that a new release come out soon!
Cheers

@Bjoernsen
Copy link

I upgraded one of my biggest Rails app (32,000 LOC) to 7.2 using your branch. All tests succeeded and up to now, I cannot find anything, that does not work.

@matthewtusker
Copy link

@rsim Are you still maintaining this project?

@dwarner It appears that this repo is mostly maintained by @yahonda. I would love to see @andynu added as a maintainer as someone who managed to contribute a lot to this Gem recently (if that's something they're interested in).

@dwarner
Copy link
Author

dwarner commented May 29, 2025

+1. Any updates on this or any sort of plans to maintain this gem?

@yahonda Are you still actively maintaining this gem?

@dwarner
Copy link
Author

dwarner commented May 29, 2025

@rsim Are you still maintaining this project?

@dwarner It appears that this repo is mostly maintained by @yahonda. I would love to see @andynu added as a maintainer as someone who managed to contribute a lot to this Gem recently (if that's something they're interested in).

I second that!

If we don't get a response, even though nothing has been posted since 2021, I might try pinging @yahonda here:
https://groups.google.com/g/oracle-enhanced

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

No branches or pull requests