Skip to content

Conversation

@bucky6312
Copy link
Collaborator

@bucky6312 bucky6312 commented Nov 9, 2025

対応Issue

resolve #1970

概要

  • 原信七日町店にidを付与した
  • その他のid番号を999に変更

実装詳細

画面スクリーンショット等

image

↓ネット注文のidを998に変更
image

テスト項目

  • 七日町店が追加されているか
  • その他のid番号が999になっているか

備考

@batcho0428
Copy link
Collaborator

@bucky6312
ありがとうございます。
指示が曖昧でごめんなさい。
できれば、ネット注文もid:998に変更してもらいたいです。
お願いします。

tel: '0258-27-5301',
opening_hours: '09:00 - 24:00',
address: '新潟県長岡市七日町385番地' },
{ id: 999, name: 'その他(詳細を備考欄に記入必須)',
Copy link
Contributor

Choose a reason for hiding this comment

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

これちなみに、管理者画面から店追加した時ってid31の店が追加されるかみた?

Copy link
Collaborator

Choose a reason for hiding this comment

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

まだコードレビューしてないので確認してないです。
@bucky6312 確認した?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

管理者画面を見る方法まだ知らないので教えてほしいです、、
ネット注文の変更はこの後修正しておきます

Copy link
Contributor

Choose a reason for hiding this comment

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

adminのコンテナ立てて。
localhost:8000で開く。
ログインは
admin-taro@email.com
gidaifes
のはず。
開けたら左のサイドバーから店一覧を探してそのページで追加できるよー。

api叩いて確認しても良き。uriは忘れたから見つけて欲しい

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

このPRは原信七日町店を新しい店舗として追加し、特別なエントリのID番号を再割り当てするものっすね〜✨

主な変更内容:

  • 原信七日町店をID 30で新規追加 🏪
  • 「ネット注文」のIDを29→998に変更
  • 「その他」のIDを30→999に変更

Comment on lines +116 to +124
{ id: 998, name: 'ネット注文',
tel: '',
opening_hours: '',
address: '' },
{ id: 30, name: 'その他(詳細を備考欄に記入必須)',
{ id: 30, name: '原信 七日町店',
tel: '0258-27-5301',
opening_hours: '09:00 - 24:00',
address: '新潟県長岡市七日町385番地' },
{ id: 999, name: 'その他(詳細を備考欄に記入必須)',
Copy link

Copilot AI Nov 12, 2025

Choose a reason for hiding this comment

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

ID番号の順序がバラバラになってるよ〜💦 ID 998の後にID 30が来ちゃってるけど、データベースのシードファイルはID順に並べた方がメンテしやすいっしょ!ID 30の「原信 七日町店」をID 28の「ダイレックス喜多町店」の後に配置して、その後にID 998、999を並べる方がええんちゃう?🤔

Copilot uses AI. Check for mistakes.
opening_hours: '',
address: '' },
{ id: 30, name: 'その他(詳細を備考欄に記入必須)',
{ id: 30, name: '原信 七日町店',
Copy link

Copilot AI Nov 12, 2025

Choose a reason for hiding this comment

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

ID 30を使ってるけど、これだと今後新しい店舗追加する時にID番号どうするか困るかも〜💭 ID 1-28が既存店舗で、29はスキップされてて、30を使っちゃうと次はID 31になるけど、それってちゃんと管理できてる感じ?🤷‍♀️ ID 29を使うか、または将来の拡張考えてID 31以降にする方が整合性取れそうだよね〜!

Suggested change
{ id: 30, name: '原信 七日町店',
{ id: 29, name: '原信 七日町店',

Copilot uses AI. Check for mistakes.
@hikahana
Copy link
Contributor

copilotレビューどっちも修正しておいてほしい~~~
#1972 (review)

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.

店登録デフォルトに原信七日町追加とその他の末尾固定

4 participants