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

Update to support Rails 7.0 #34

Merged
merged 15 commits into from
Aug 16, 2024
Merged

Update to support Rails 7.0 #34

merged 15 commits into from
Aug 16, 2024

Conversation

davkutalek
Copy link
Contributor

@davkutalek davkutalek commented Aug 13, 2024

Tested with existing 6.1 application and a 7.0 app.

Once this is merged I'll create a release and push it to ruby gems.

Base automatically changed from ruby-3-1 to main August 13, 2024 21:51
@davkutalek davkutalek requested review from timlkelly and a team August 14, 2024 20:10
@davkutalek davkutalek changed the title update to rails 7.0 support, remove depreciated api references Update to support Rails 7.0 Aug 14, 2024
Copy link

@timlkelly timlkelly left a comment

Choose a reason for hiding this comment

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

Initial comments and testing. I set my local knockoff to this branch and then set ENABLE_REPLICAS=true in my .env.development.local and received this error:

F, [2024-08-14T15:26:35.931451 #85048] FATAL -- : [d32d9ef4-91fd-43be-914a-db4209c31fc7] [Chrome 127.0.0.0] [::1] [dd.env=development dd.service=handshake dd.trace_id=3860344763007303586 dd.span_id=0 ddsource=ruby]
[d32d9ef4-91fd-43be-914a-db4209c31fc7] [Chrome 127.0.0.0] [::1] [dd.env=development dd.service=handshake dd.trace_id=3860344763007303586 dd.span_id=0 ddsource=ruby] NameError (uninitialized constant Knockoff::ActiveRecord::Base

      open_transactions = run_on(:primary) { ActiveRecord::Base.connection.open_transactions }
                                                         ^^^^^^
Did you mean?  Base64):
[d32d9ef4-91fd-43be-914a-db4209c31fc7] [Chrome 127.0.0.0] [::1] [dd.env=development dd.service=handshake dd.trace_id=3860344763007303586 dd.span_id=0 ddsource=ruby]
[d32d9ef4-91fd-43be-914a-db4209c31fc7] [Chrome 127.0.0.0] [::1] [dd.env=development dd.service=handshake dd.trace_id=3860344763007303586 dd.span_id=0 ddsource=ruby] lib/middleware/next/transition_redirections.rb:25:in `call'

CHANGELOG.md Show resolved Hide resolved
knockoff.gemspec Show resolved Hide resolved
README.md Show resolved Hide resolved
@davkutalek davkutalek requested a review from timlkelly August 15, 2024 22:25
@davkutalek davkutalek marked this pull request as ready for review August 15, 2024 22:25
README.md Outdated
@@ -17,7 +17,29 @@ A gem for easily using read replicas.

## Supported Versions

Knockoff supports Rails 4, 5 and 6
| Knockoff version: | 1.0 | 1.1.0 | 1.1.1 | 1.4 | 1.5 |
| -------- | ------- | -------- | ------- | -------- |

Choose a reason for hiding this comment

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

I think you need one more divider. There are six columns in total. Right now, it looks like this

image

@davkutalek davkutalek requested a review from timlkelly August 16, 2024 16:28
@davkutalek davkutalek merged commit bd1cfdf into main Aug 16, 2024
3 checks passed
@davkutalek davkutalek deleted the rails-7 branch August 16, 2024 18:28
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