火器使用申請のUI作成 #1925
Open
火器使用申請のUI作成 #1925
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
izuizu0424
reviewed
Jun 3, 2025
Collaborator
There was a problem hiding this comment.
ありがとうございます！デザインレビューのみです！
何も入力していないとき，すでに値が入っていたんですがこれは実装の段階で改善されますかね？
火気使用申請で「いいえ」を選択するとき以下のようなListを表示してほしいです！
すみませんデザインから間違ってました...
火器ではなく火気です
- 火気使用申請
- 火気を使用しますか？
- 火気の名称
- 火気の台数
- 火気を毎日テントから持ち帰ることができますか
- 火気は毎日持って帰ることができない場合を除き，基本的に持ち帰ってくだい。
火気をテント内に残す行為は火事の原因になります。
この部分を注意書き含めデザインに沿って変更お願いします
現在の実装
figma
登録を押してもカード化しないのでカード化までお願いします！
登録出来たらトースト表示をお願いします！
added 2 commits
June 3, 2025 21:04
Collaborator
Author
|
追加しました |
Collaborator
Author
Collaborator
Author
TkymHrt
reviewed
Jun 20, 2025
| required | ||
| options={[ | ||
| { | ||
| id: 0, |
Member
There was a problem hiding this comment.
マジックナンバーかもです。
// ex
const PLACEHOLDER_OPTION_ID = 0;
Contributor
There was a problem hiding this comment.
下のも含め、constantに入れるようにしました
Comment on lines
102
to
109
| <p className="max-w-[400px] break-words text-xs text-[#484848]"> | ||
| 火気は毎日持って帰ることができない場合を除き、基本的に持ち帰ってください。 | ||
| <br /> | ||
| 火気はテント内に残す行為は火事の原因になります。 | ||
| </p> | ||
| <p className="max-w-[400px] break-words text-xs text-[#484848]"> | ||
| いいえを押した場合は火気の備考欄に理由を記載して下さい。 | ||
| </p> |
Member
There was a problem hiding this comment.
constant.ts にまとめても良いかもです。
// ex
const FIRE_EQUIPMENT_INSTRUCTIONS = {
TAKEAWAY_NOTICE: '...',
REMARK_NOTICE: '...'
};
（ label 等もまとめられそう）
Contributor
|
コンフリクトの解消、API型とFORM型の命名規則の統一、UIの調整を対応しました |
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.
resolve #1908
概要
実装詳細
このプルリクエストでは、コンポーネント、フック、検証スキーマなど、消防設備申請を管理するための新機能が導入されます。これらの変更により、ユーザーは消防設備申請フォームを送信、編集、検証できるようになり、登録状況と期限に基づく条件付きロジックもサポートされます。
火器使用申請機能の実装:
コアコンポーネント:
FireEquipment.tsx: 期限と登録状況に応じた条件付きレンダリングを含む、消防設備申請のメインコンポーネントを実装します。FireEquipmentForm.tsx: 消防設備の詳細を送信するための詳細なフォームを提供します。名前、数量、燃料の種類、用途、備考などのフィールドが含まれます。FireEquipmentFormView.tsx: 登録ステータスと編集モードを処理するロジックでフォームをラップします。データ処理と検証:
hooks.ts: 消防設備の状態、フォームの送信、検証ロジックを管理するためのカスタムフックを追加します。登録済みおよび未登録の消防設備シナリオのサポートが含まれます。 [1] [2]schema.ts:消防設備フォームのZod検証スキーマを定義します。これには、持ち帰り状況に基づくコメントの条件付き検証が含まれます。ユーティリティと定数:
constant.ts: 消防設備フォームのフィールドとラジオボタンのオプションに列挙型と定数を導入します。 [1] [2]既存のコードベースとの統合:
Storybook とコンポーネントのインデックス作成:
FireEquipment.stories.tsx: テストとドキュメント作成を容易にするために、消防設備コンポーネントの Storybook ストーリーを追加します。index.ts: アプリケーション全体で使用するために新しいコンポーネントをエクスポートします。 [1] [2]ホームページ統合:
home/index.tsx:消防設備コンポーネントをホームページに統合し、ユーザーがその機能にアクセスできるようにします。 [1] [2]フォームの機能強化:
TextArea.tsx:フォーム フィールドの使いやすさを向上させるために、カスタム必須メッセージのサポートが追加されました。 [1] [2]Radio.tsx:Option型を再利用のためにエクスポートしますラジオボタンの設定で。これらの変更により、消防設備アプリケーションを管理するための堅牢なシステムが導入され、検証、条件付きレンダリング、既存のワークフローとの統合を通じてユーザーエクスペリエンスが向上します。
画面スクリーンショット等
テスト項目
備考