fix: Deleted pages appear in the page tree #9337
Conversation
apps/app/src/stores/page-listing.tsx
Outdated
| @@ -197,7 +197,7 @@ export const useSWRxPageAncestorsChildren = ( | |||
| assert(keyMatcherForPageTree(key)); | |||
| } | |||
|
|
|||
| return useSWRImmutable( | |||
| return useSWR( | |||
There was a problem hiding this comment.
useSWRxPageAncestorsChildren の自動 revalidation を許すと、API を叩く回数が増えるのを心配している。
https://swr.vercel.app/ja/docs/revalidation#disable-automatic-revalidations
関係ないトリガーはもう少し抑制できないだろうか。
There was a problem hiding this comment.
useSWRxPageAncestorsChildren の useSWR を Immutable に戻しました。
TreeItemLayout.tsx の受け取る PageTree の情報は、上位から受け取る useSWRxPageAncestorsChildren の childrenA と 41 行目の useSWRxPageChildren の childrenB の2つがありました。/trash に遷移した際、childrenA は削除したものが残っており、childrenB はちゃんと消えていました。常に childrenB の情報のほうが正しかったので、childrenA を受け取るのを辞める形で変更しました。
|
| @@ -36,7 +36,7 @@ export const TreeItemLayout: FC<TreeItemLayoutProps> = (props) => { | |||
|
|
|||
| const { page, children } = itemNode; | |||
|
|
|||
| const [currentChildren, setCurrentChildren] = useState(children); | |||
| const [currentChildren, setCurrentChildren] = useState<ItemNode[]>(children); | |||
There was a problem hiding this comment.
メモ
こちらのデグレに対応
https://redmine.weseek.co.jp/issues/157808
mergify
bot
deleted the
fix/111603-126863-pagetree-display-the-page-that-is-deleted
branch
Summary
Task
Note
垂れ流し
TreeItemLayout.tsxの変更は、上位からくるuseSWRxPageAncestorsChildrenの children をcurrentChildrenの更新に使わないようにする変更。