migrate_adding_id_to_yaml で created_at も書き出すよう修正
#1772
Conversation
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
2 tasks
yasulab
reviewed
Dec 12, 2025
| new_dojo = {} | ||
| new_dojo['id'] = d.id | ||
| #new_dojo['order'] = d.order # ID の直後に固定させる場合の例 | ||
| new_dojo['order'] = d.order |
Member
There was a problem hiding this comment.
今回の title/description を読む限りではココのコメントアウトの解除は不要そうかなと思いましたが、もし何か解除した理由などあれば description に記載いただけると助かります...!! 🙏 ✨
Contributor
Author
There was a problem hiding this comment.
@yasulab コメントありがとうございます！
YAML 内の既存の記述に影響を与えない（記載内容の並び順を既存と揃える）ため 、new_dojo['order'] = d.order のコメントアウトを解除しています。
もし、既存の記載内容の並び順が変わってしまっても問題ないようでしたら、2. のようにコメントアウトを解除しない方が、記述もその後の処理もスッキリしそうと感じたのですが、どちらが良さそうでしょうか 👀💭
new_dojo['order'] = d.orderのコメントアウトを解除する- YAMLの記載は既存の並び順の通り
- id
- order
- created_at
- ...
- YAMLの記載は既存の並び順の通り
new_dojo['order'] = d.orderのコメントアウトを解除しない- YAMLの記載順は以下になる
- id
- created_at
- order
- ...
- 既存の記述も記載順が入れ替わる(内容は変わらない
- YAMLの記載順は以下になる
Member
There was a problem hiding this comment.
@rakuda-san-desu なるほどです! 理由がどこにも記載されておらずコメント解除のみ行われていた点 が気になった部分ですので、であればその理由を description に記載していただけると助かります! (＞人＜ )✨
Contributor
Author
There was a problem hiding this comment.
description に追記しました！
今後も理由まで記載するようにしていきます📝
ご確認ありがとうございます🙏✨
Contributor
|
タスクを実行すると、 🔽 出力結果 - id: 349
order: '011002'
created_at: '2025-12-16'
name: サンプル
prefecture_id: 1
logo: "/img/dojos/default.webp"
url: https://example.com/
description: 札幌市で月2回開催
tags:
- Scratch
- micro:bit
- Arduino
- ラズベリーパイ
- 電子工作 |
Contributor
Author
|
@nacchan99 動作確認ありがとうございます🙏✨ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
close: #1770
やったこと
migrate_adding_id_to_yamlを実行したとき、idと一緒にcreated_atも書き出すよう修正しました🛠️メモ:
YAML 内の既存の記述に影響を与えない（記載内容の並び順を既存と揃える）ため 、
new_dojo['order'] = d.orderのコメントアウトを解除しています！確認したこと
idとcreated_atが書き出される確認方法
db/dojos.ymlに以下のサンプルを入力$ bundle exec rails dojos:update_db_by_yamlを実行$ bundle exec rails dojos:migrate_adding_id_to_yamlを実行idとcreated_atが書き出されるMEMO: 確認後、サンプルで作成されたデータは削除しておく