Skip to content

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented May 20, 2025

関連ページまとめ


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

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

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 nalabjp and nanophate 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