-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Conversation
@@ -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 |
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.
シリアライズ方法を指定する必要があったため、既存の DB 内にあるデータ構造を確認し、当該データ構造(YAML)を明示的に指定しました。
セキュリティ対応も考えると早めに cf. https://rubyonrails.org/maintenance ![]() |
無事デプロイを確認できました!🚀✨ |
アップデートありがとございます!!ナイスです😊✨ |
@yasulab |
# | ||
# 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`. |
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.
@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
to7.1
.
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.
こちら 67e2770 で対応しました!🛠💨 コメントありがとうございました...!! (>人< )✨
config/initializers/new_framework_defaults_7_1.rb
には you can remove this file
とありましたが、今後 default の設定例を確認したいときに便利そう + 残しておいても特に影響は無さそうだったので、当該ファイルは一旦残してあります! 📝 ✅
関連ページまとめ
cf. Railsガイド - Railsアップグレードガイド
cf. Ruby on Rails - Maintenance Policy
関連PR (時系列順):
<-- 今ココ
本 PR で行う作業 (時系列順)
(※ 変更の詳細はコミットメッセージでまとめています)
bin/rails app:update
を実行し、発生した Conflict を1つずつ対応するconfig/initializers/new_framework_defaults_7_1.rb
) に沿って、設定を1つずつ ON にしていく