From 8dd3bc0ac8c7035edec7974e76f4e49d7e401ab5 Mon Sep 17 00:00:00 2001 From: crupest Date: Fri, 31 Jul 2020 00:14:32 +0800 Subject: Update post list when create or delete post. --- .../src/app/timeline/TimelinePageTemplate.tsx | 44 +++++----------------- 1 file changed, 10 insertions(+), 34 deletions(-) (limited to 'Timeline/ClientApp/src/app/timeline/TimelinePageTemplate.tsx') diff --git a/Timeline/ClientApp/src/app/timeline/TimelinePageTemplate.tsx b/Timeline/ClientApp/src/app/timeline/TimelinePageTemplate.tsx index 88066b76..a68d08c6 100644 --- a/Timeline/ClientApp/src/app/timeline/TimelinePageTemplate.tsx +++ b/Timeline/ClientApp/src/app/timeline/TimelinePageTemplate.tsx @@ -14,7 +14,7 @@ import { usePostList, } from '../data/timeline'; -import { TimelinePostInfoEx, TimelineDeleteCallback } from './Timeline'; +import { TimelineDeleteCallback } from './Timeline'; import { TimelineMemberDialog } from './TimelineMember'; import TimelinePropertyChangeDialog from './TimelinePropertyChangeDialog'; import { TimelinePageTemplateUIProps } from './TimelinePageTemplateUI'; @@ -55,7 +55,7 @@ export default function TimelinePageTemplate< undefined ); - const rawPosts = usePostList(timeline?.name); + const postListState = usePostList(timeline?.name); const [error, setError] = React.useState(undefined); @@ -182,43 +182,19 @@ export default function TimelinePageTemplate< const onDelete: TimelineDeleteCallback = React.useCallback( (index, id) => { - service.deletePost(name, id).subscribe( - () => { - // TODO: Remove this. - setPosts((oldPosts) => - without( - oldPosts as TimelinePostInfoEx[], - (oldPosts as TimelinePostInfoEx[])[index] - ) - ); - }, - () => { - pushAlert({ - type: 'danger', - message: t('timeline.deletePostFailed'), - }); - } - ); + service.deletePost(name, id).subscribe(null, () => { + pushAlert({ + type: 'danger', + message: t('timeline.deletePostFailed'), + }); + }); }, [service, name, t] ); const onPost: TimelinePostSendCallback = React.useCallback( (req) => { - return service - .createPost(name, req) - .pipe( - map((newPost) => { - // TODO: Remove this. - setPosts((oldPosts) => - concat(oldPosts as TimelinePostInfoEx[], { - ...newPost, - deletable: true, - }) - ); - }) - ) - .toPromise(); + return service.createPost(name, req).toPromise().then(); }, [service, name] ); @@ -245,7 +221,7 @@ export default function TimelinePageTemplate<