[WEB-1116] fix: current version not displaying the latest content (#5573)

* fix: current version sync

* chore: update read only editor ref type
This commit is contained in:
Aaryan Khandelwal 2024-09-10 16:13:20 +05:30 committed by GitHub
parent 1da97d5814
commit 8acb60baef
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 32 additions and 25 deletions

View file

@ -2,7 +2,7 @@ import { useEffect, useRef, useState } from "react";
import { observer } from "mobx-react"; import { observer } from "mobx-react";
import { useSearchParams } from "next/navigation"; import { useSearchParams } from "next/navigation";
// editor // editor
import { EditorRefApi, useEditorMarkings } from "@plane/editor"; import { EditorReadOnlyRefApi, EditorRefApi, useEditorMarkings } from "@plane/editor";
// types // types
import { TPage } from "@plane/types"; import { TPage } from "@plane/types";
// ui // ui
@ -35,7 +35,7 @@ export const PageRoot = observer((props: TPageRootProps) => {
const [isVersionsOverlayOpen, setIsVersionsOverlayOpen] = useState(false); const [isVersionsOverlayOpen, setIsVersionsOverlayOpen] = useState(false);
// refs // refs
const editorRef = useRef<EditorRefApi>(null); const editorRef = useRef<EditorRefApi>(null);
const readOnlyEditorRef = useRef<EditorRefApi>(null); const readOnlyEditorRef = useRef<EditorReadOnlyRefApi>(null);
// router // router
const router = useAppRouter(); const router = useAppRouter();
// search params // search params
@ -89,11 +89,15 @@ export const PageRoot = observer((props: TPageRootProps) => {
editorRef.current?.clearEditor(); editorRef.current?.clearEditor();
editorRef.current?.setEditorValue(descriptionHTML); editorRef.current?.setEditorValue(descriptionHTML);
}; };
const currentVersionDescription = isContentEditable
? editorRef.current?.getHTML()
: readOnlyEditorRef.current?.getHTML();
return ( return (
<> <>
<PageVersionsOverlay <PageVersionsOverlay
activeVersion={version} activeVersion={version}
currentVersionDescription={currentVersionDescription ?? null}
editorComponent={PagesVersionEditor} editorComponent={PagesVersionEditor}
fetchAllVersions={async (pageId) => { fetchAllVersions={async (pageId) => {
if (!workspaceSlug || !projectId) return; if (!workspaceSlug || !projectId) return;

View file

@ -7,20 +7,20 @@ import { IUserLite, TPageVersion } from "@plane/types";
// plane ui // plane ui
import { Loader } from "@plane/ui"; import { Loader } from "@plane/ui";
// hooks // hooks
import { useMember, useMention, usePage, useUser } from "@/hooks/store"; import { useMember, useMention, useUser } from "@/hooks/store";
import { usePageFilters } from "@/hooks/use-page-filters"; import { usePageFilters } from "@/hooks/use-page-filters";
// plane web hooks // plane web hooks
import { useIssueEmbed } from "@/plane-web/hooks/use-issue-embed"; import { useIssueEmbed } from "@/plane-web/hooks/use-issue-embed";
type Props = { export type TVersionEditorProps = {
activeVersion: string | null; activeVersion: string | null;
currentVersionDescription: string | null;
isCurrentVersionActive: boolean; isCurrentVersionActive: boolean;
pageId: string;
versionDetails: TPageVersion | undefined; versionDetails: TPageVersion | undefined;
}; };
export const PagesVersionEditor: React.FC<Props> = observer((props) => { export const PagesVersionEditor: React.FC<TVersionEditorProps> = observer((props) => {
const { activeVersion, isCurrentVersionActive, pageId, versionDetails } = props; const { activeVersion, currentVersionDescription, isCurrentVersionActive, versionDetails } = props;
// params // params
const { workspaceSlug, projectId } = useParams(); const { workspaceSlug, projectId } = useParams();
// store hooks // store hooks
@ -29,7 +29,6 @@ export const PagesVersionEditor: React.FC<Props> = observer((props) => {
getUserDetails, getUserDetails,
project: { getProjectMemberIds }, project: { getProjectMemberIds },
} = useMember(); } = useMember();
const currentPageDetails = usePage(pageId);
// derived values // derived values
const projectMemberIds = projectId ? getProjectMemberIds(projectId.toString()) : []; const projectMemberIds = projectId ? getProjectMemberIds(projectId.toString()) : [];
const projectMemberDetails = projectMemberIds?.map((id) => getUserDetails(id) as IUserLite); const projectMemberDetails = projectMemberIds?.map((id) => getUserDetails(id) as IUserLite);
@ -92,7 +91,7 @@ export const PagesVersionEditor: React.FC<Props> = observer((props) => {
</div> </div>
); );
const description = isCurrentVersionActive ? currentPageDetails.description_html : versionDetails?.description_html; const description = isCurrentVersionActive ? currentVersionDescription : versionDetails?.description_html;
if (description === undefined || description?.trim() === "") return null; if (description === undefined || description?.trim() === "") return null;
return ( return (

View file

@ -6,17 +6,15 @@ import { TriangleAlert } from "lucide-react";
import { TPageVersion } from "@plane/types"; import { TPageVersion } from "@plane/types";
// plane ui // plane ui
import { Button, setToast, TOAST_TYPE } from "@plane/ui"; import { Button, setToast, TOAST_TYPE } from "@plane/ui";
// components
import { TVersionEditorProps } from "@/components/pages";
// helpers // helpers
import { renderFormattedDate, renderFormattedTime } from "@/helpers/date-time.helper"; import { renderFormattedDate, renderFormattedTime } from "@/helpers/date-time.helper";
type Props = { type Props = {
activeVersion: string | null; activeVersion: string | null;
editorComponent: React.FC<{ currentVersionDescription: string | null;
activeVersion: string | null; editorComponent: React.FC<TVersionEditorProps>;
isCurrentVersionActive: boolean;
pageId: string;
versionDetails: TPageVersion | undefined;
}>;
fetchVersionDetails: (pageId: string, versionId: string) => Promise<TPageVersion | undefined>; fetchVersionDetails: (pageId: string, versionId: string) => Promise<TPageVersion | undefined>;
handleClose: () => void; handleClose: () => void;
handleRestore: (descriptionHTML: string) => Promise<void>; handleRestore: (descriptionHTML: string) => Promise<void>;
@ -25,8 +23,16 @@ type Props = {
}; };
export const PageVersionsMainContent: React.FC<Props> = observer((props) => { export const PageVersionsMainContent: React.FC<Props> = observer((props) => {
const { activeVersion, editorComponent, fetchVersionDetails, handleClose, handleRestore, pageId, restoreEnabled } = const {
props; activeVersion,
currentVersionDescription,
editorComponent,
fetchVersionDetails,
handleClose,
handleRestore,
pageId,
restoreEnabled,
} = props;
// states // states
const [isRestoring, setIsRestoring] = useState(false); const [isRestoring, setIsRestoring] = useState(false);
const [isRetrying, setIsRetrying] = useState(false); const [isRetrying, setIsRetrying] = useState(false);
@ -112,8 +118,8 @@ export const PageVersionsMainContent: React.FC<Props> = observer((props) => {
<div className="pt-8 h-full overflow-y-scroll vertical-scrollbar scrollbar-sm"> <div className="pt-8 h-full overflow-y-scroll vertical-scrollbar scrollbar-sm">
<VersionEditor <VersionEditor
activeVersion={activeVersion} activeVersion={activeVersion}
currentVersionDescription={currentVersionDescription}
isCurrentVersionActive={isCurrentVersionActive} isCurrentVersionActive={isCurrentVersionActive}
pageId={pageId}
versionDetails={versionDetails} versionDetails={versionDetails}
/> />
</div> </div>

View file

@ -2,18 +2,14 @@ import { observer } from "mobx-react";
// plane types // plane types
import { TPageVersion } from "@plane/types"; import { TPageVersion } from "@plane/types";
// components // components
import { PageVersionsMainContent, PageVersionsSidebarRoot } from "@/components/pages"; import { PageVersionsMainContent, PageVersionsSidebarRoot, TVersionEditorProps } from "@/components/pages";
// helpers // helpers
import { cn } from "@/helpers/common.helper"; import { cn } from "@/helpers/common.helper";
type Props = { type Props = {
activeVersion: string | null; activeVersion: string | null;
editorComponent: React.FC<{ currentVersionDescription: string | null;
activeVersion: string | null; editorComponent: React.FC<TVersionEditorProps>;
isCurrentVersionActive: boolean;
pageId: string;
versionDetails: TPageVersion | undefined;
}>;
fetchAllVersions: (pageId: string) => Promise<TPageVersion[] | undefined>; fetchAllVersions: (pageId: string) => Promise<TPageVersion[] | undefined>;
fetchVersionDetails: (pageId: string, versionId: string) => Promise<TPageVersion | undefined>; fetchVersionDetails: (pageId: string, versionId: string) => Promise<TPageVersion | undefined>;
handleRestore: (descriptionHTML: string) => Promise<void>; handleRestore: (descriptionHTML: string) => Promise<void>;
@ -26,6 +22,7 @@ type Props = {
export const PageVersionsOverlay: React.FC<Props> = observer((props) => { export const PageVersionsOverlay: React.FC<Props> = observer((props) => {
const { const {
activeVersion, activeVersion,
currentVersionDescription,
editorComponent, editorComponent,
fetchAllVersions, fetchAllVersions,
fetchVersionDetails, fetchVersionDetails,
@ -51,6 +48,7 @@ export const PageVersionsOverlay: React.FC<Props> = observer((props) => {
> >
<PageVersionsMainContent <PageVersionsMainContent
activeVersion={activeVersion} activeVersion={activeVersion}
currentVersionDescription={currentVersionDescription}
editorComponent={editorComponent} editorComponent={editorComponent}
fetchVersionDetails={fetchVersionDetails} fetchVersionDetails={fetchVersionDetails}
handleClose={handleClose} handleClose={handleClose}