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

Bump to rails version 8.0.0 #4714

Merged
merged 3 commits into from
Dec 3, 2024
Merged

Bump to rails version 8.0.0 #4714

merged 3 commits into from
Dec 3, 2024

Conversation

gms-gs
Copy link
Contributor

@gms-gs gms-gs commented Nov 26, 2024

Context

Ticket: https://trello.com/c/uEmppsew/262-upgrade-to-rails-8
As part of keeping the application up to date we decided to upgrade to Rails 8.0.0

Changes proposed in this pull request

Ran rails app:update which created/edited a lot of files.

  • Updates many auto-generated config files, some changes just shift the content around to make it easier for future upgrades.
  • Other small changes to the code to work with rails 8.
  • Fix start_date for the EditCourseForm. We can't pass Structs when updating records in rails 8 any more, we need to pass nil not Struct(day: 1, month: nil, year: nil) when creating or updating records.

Guidance to review

Play through the review app and see if things are working as expected, however we can hopefully rely on all tests passing as a good indicator of a successful upgrade.

@gms-gs gms-gs self-assigned this Nov 26, 2024
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch from 6dde0e7 to 1973f42 Compare November 26, 2024 13:52
Gemfile Outdated Show resolved Hide resolved
@gms-gs gms-gs changed the base branch from main to gs/feature/remove-canonical-rails-gem November 26, 2024 13:56
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 3 times, most recently from f0e7385 to 439dd11 Compare November 26, 2024 15:05
@gms-gs
Copy link
Contributor Author

gms-gs commented Nov 26, 2024

Upgrade to Rails 8

Base automatically changed from gs/feature/remove-canonical-rails-gem to main November 27, 2024 12:07
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 2 times, most recently from 238ec68 to ff2eb58 Compare November 27, 2024 12:12
@gms-gs gms-gs added the deploy A Review App will be created for PRs with this label label Nov 27, 2024
@gms-gs gms-gs marked this pull request as ready for review November 30, 2024 11:09
@gms-gs gms-gs requested a review from a team as a code owner November 30, 2024 11:09
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 2 times, most recently from fcfa052 to f24e479 Compare November 30, 2024 14:48
app/controllers/publish/providers_controller.rb Outdated Show resolved Hide resolved
bin/dev Outdated Show resolved Hide resolved
bin/setup Show resolved Hide resolved
config/initializers/new_framework_defaults_8_0.rb Outdated Show resolved Hide resolved
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 3 times, most recently from 3a033fa to d84bdf4 Compare December 2, 2024 11:23
@gms-gs gms-gs requested a review from Nitemaeric December 2, 2024 11:23
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 9 times, most recently from 14d9b20 to b34e6fb Compare December 2, 2024 14:52
Copy link
Contributor

@inulty-dfe inulty-dfe left a comment

Choose a reason for hiding this comment

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

Looks ok, I've clicked about in the review app.

Are we not including the options in the environment files that are new in Rails 8?

bin/thrust Outdated Show resolved Hide resolved
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 2 times, most recently from 209f100 to 98ddd5a Compare December 2, 2024 17:01
We can't pass Structs when updating records in rails 8 any more, we need
to pass nil not Struct(day: 1, month: nil, year: nil) when creating or
updating records.
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch 3 times, most recently from 06dc371 to c2b0ca1 Compare December 2, 2024 17:26
Rails 8 added a new attribute that AccreditingProviderEnrichment now needs to ignore
context_for_validation.
@gms-gs gms-gs force-pushed the bump-rails-v-8.0.0 branch from c2b0ca1 to 2a29f36 Compare December 2, 2024 17:36
@gms-gs gms-gs merged commit c4f6819 into main Dec 3, 2024
19 checks passed
@gms-gs gms-gs deleted the bump-rails-v-8.0.0 branch December 3, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy A Review App will be created for PRs with this label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants