Skip to content

Upgrade Ruby on Rails: 7.0 -> 7.1 #1686

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 16 commits into from
May 20, 2025
Merged

Upgrade Ruby on Rails: 7.0 -> 7.1 #1686

merged 16 commits into from
May 20, 2025

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented May 20, 2025

関連ページまとめ


本 PR で行う作業 (時系列順)

(※ 変更の詳細はコミットメッセージでまとめています)

@@ -10,7 +10,7 @@ class Dojo < ApplicationRecord
has_many :dojo_event_services, dependent: :destroy
has_many :event_histories, dependent: :destroy

serialize :tags
serialize :tags, coder: YAML
Copy link
Member Author

@yasulab yasulab May 20, 2025

Choose a reason for hiding this comment

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

シリアライズ方法を指定する必要があったため、既存の DB 内にあるデータ構造を確認し、当該データ構造(YAML)を明示的に指定しました。

@yasulab yasulab requested review from nanophate and nalabjp May 20, 2025 12:21
@yasulab
Copy link
Member Author

yasulab commented May 20, 2025

セキュリティ対応も考えると早めに Rails 7.1.x にアップグレードした方が良さそうなので、一旦マージしちゃいます! 何か気になる点などあれば後日コメントなどでフィードバックいただければ...!! 🙏 ✨ cc/ @nanophate @nalabjp

cf. https://rubyonrails.org/maintenance

image

@yasulab yasulab merged commit dfb61dd into main May 20, 2025
4 checks passed
@yasulab yasulab deleted the upgrade-to-rails-7-1 branch May 20, 2025 13:53
@yasulab
Copy link
Member Author

yasulab commented May 20, 2025

無事デプロイを確認できました!🚀✨

@nanophate
Copy link
Member

アップデートありがとございます!!ナイスです😊✨

yasulab added a commit that referenced this pull request May 20, 2025
@nalabjp
Copy link
Member

nalabjp commented May 21, 2025

@yasulab
load_defaults 7.1するとnew_framework_defaults_7_1.rb内の設定を全て有効にしたのと同義なんですが、意図通りでしょうか?
設定の詳細それぞれは把握できていないのですが、コメントアウトされたままの設定が残っているので気になりました。
意図通りであるという事であればnew_framework_defaults_7_1.rbは削除してしたって良いと思いました。

#
# Uncomment each configuration one by one to switch to the new default.
# Once your application is ready to run with all new defaults, you can remove
# this file and set the `config.load_defaults` to `7.1`.
Copy link
Member Author

@yasulab yasulab May 21, 2025

Choose a reason for hiding this comment

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

@nalabjp コメントありがとうございます!確かにココに書いてありますね... 💦 最初の1行目の通りに進めていったんですが、

Uncomment each configuration one by one to switch to the new default.

以下にある通り、うまく動いたら次の設定をするだけでよかったんですね 😂 後で直します!! 🛠💨✨

Once your application is ready to run with all new defaults, you can remove this file and set the config.load_defaults to 7.1.

Copy link
Member Author

Choose a reason for hiding this comment

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

こちら 67e2770 で対応しました!🛠💨 コメントありがとうございました...!! (>人< )✨

config/initializers/new_framework_defaults_7_1.rb には you can remove this file とありましたが、今後 default の設定例を確認したいときに便利そう + 残しておいても特に影響は無さそうだったので、当該ファイルは一旦残してあります! 📝 ✅

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.

3 participants