-
Notifications
You must be signed in to change notification settings - Fork 12
Upgrade to Rails 8.1 beta #140
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
Merged
Merged
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d659332
Rails 7.0 upgrade - wip - rails admin for enum models issues
marc 5f836af
Rails 7.1 upgrade. wip - tests passed but there might be a seeding issue
marc eecaef4
Handle missing/problem images with Rails 7.1 stricter sprockets assets
johnpaulashenfelter 6339d65
Fixed: concurrent_ruby issue doesn't exist in this version of ruby an…
johnpaulashenfelter 8dacde4
Remove unused initializers.
marc 3452874
Rails 7.2.2.2 upgrade. Tests are passing.
marc e5b875d
Add a missing image for content, generated with ChatGPT
johnpaulashenfelter ca972bb
Rails 8.0 upgrade. All tests are passing.
marc a3dbaa4
Rails 8.1.0.beta1 upgrade. All tests are passing.
marc 612994d
Merge branch 'main' into feature/rails81beta
elasticspoon 522a9cc
Attempt to fix CI error related to Rails Admin.
marc 126682f
Attempt to fix failing CI tests. (Passes locally in docker.)
marc bbf8dc0
Testing alternate config for CI tests.
marc 1658f3d
Add libyaml-dev for staging/prod docker.
marc 41b20d3
Fix Dockerfile bundle install.
marc de3e84d
feat: remove rails admin
elasticspoon 9357fae
fix: registration spec
elasticspoon 77786c5
fix: event spec
elasticspoon 6624e51
fix: gemfile
elasticspoon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wasn't this already in there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think trilogy was indirectly loaded as a dependency for activerecord-trilogy-adapter. That is no longer needed as of rails 7.1+, so I added just the trilogy adapter.