[dashboard] form周りの改善(valibotの使用, contextの利用)

eyemono-moe commented 3 months ago

なぜやるか

fix #902, fix #899

「modular formsのコンポーネントで自動生成される formのstore制御用のonInput, onChange, onBlur」と、独自のform制御 (kobalteのradio groupでbooleanが値として使えないから文字列↔booleanの変換を挟んだり, ビルドタイプの設定のために"runtime"/"static"と"buildpack"/"cmd"/"docker"の2*3のradio groupの入力値を組み合わせて"runtimeCmd"に変換したりしてる部分とか)がかみ合っていないために複数バグが存在していた。(#902, #899) https://github.com/traPtitech/NeoShowcase/issues/902#issuecomment-2228305861

やったこと

独自のform制御を削除
- 代わりにvalibotによるバリデーション, 変換処理を行うようにしました
- 902, #899 が修正されています
各formのcontext化
- props drillingの低減を目的としています
componentsの整理
- 新コンポーネントについてはfeature毎にディレクトリを切る形にしました
- 既存のコンポーネントのディレクトリ移動はしていません(別PRで整理したい)
DiscardChanges buttonの一部削除
- ref: https://q.trap.jp/messages/019169c1-0377-77db-b1e3-7cab08d166db
- 環境変数設定画面のみ残しています(一度しか表示されない系のsecretを誤って消してしまった時などに面倒だと考えたため)
form, comboboxの修正により #827 が修正されています

やらなかったこと

既存のコンポーネントのディレクトリ整理

資料

github-actions[bot] commented 3 months ago

Preview (prod backend + PR dashboard) → https://938.ns-preview.trapti.tech/

cp-20 commented 2 months ago

(まだ中身見てなくてdescription読んだだけなんですが) #827 が修正されてるなら fix #827 もしておくと良さそうです？

traPtitech / NeoShowcase