-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
以下、#175に対する山下さんのコメント
ーーー
概ねできているので、修正は後続チケットに回しましょう。
いくつか気になる点があったので、報告いたします。
page.tsxをクライアントコンポーネントにしない
ユーザグループが存在する場合で、かつ、権限外の人間が「編集」しようとした場合に、編集そのものはできてしまう
存在しないユーザグループを編集しようとした場合に、読み込み中のまま止まる(閲覧ならダッシュボードに遷移されるのでOK)
ダッシュボードにredirectは少しわかりづらいので、専用のエラーページ(というかパス)を用意
1.についてはuseParamsやuseEffectを使う都合上「クライアントコンポーネント」にしたと推察していますが、
パスパラメータは、クライアントコンポーネントでも取得可能です。
(front/src/app/(withsidebar)/event/eventedit/[eventId]/page.tsxを参考に)
また、useEffectを使う理由があまりわからなかったです。
あくまでクライアントからHTMLを返すだけなので、普通に関数を実行して普通にif文で制御する方向で問題ないと考えています。
2., 3.については、多分修正漏れだと思います。閲覧ではうまくいっているので。
4.については後に回しましょう。
Metadata
Metadata
Assignees
Labels
No labels