Skip to content

【ユーザーグループ】詳細、編集に関する修正 #212

@YasunariIguchi

Description

@YasunariIguchi

以下、#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

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions