diff --git a/packages/client/src/components/document/fullscreen/index.tsx b/packages/client/src/components/document/fullscreen/index.tsx index 9fb8a37a..d7361ae0 100644 --- a/packages/client/src/components/document/fullscreen/index.tsx +++ b/packages/client/src/components/document/fullscreen/index.tsx @@ -74,11 +74,14 @@ export const DocumentFullscreen: React.FC = ({ data }) => { const [isDrawing, toggleDrawing] = useToggle(false); const [cover, setCover] = useState(''); - const editor = useEditor({ - editable: false, - extensions: CollaborationKit.filter((ext) => ['title', 'doc'].indexOf(ext.name) < 0).concat(Document), - content: { type: 'doc', content: [] }, - }); + const editor = useEditor( + { + editable: false, + extensions: CollaborationKit.filter((ext) => ['title', 'doc'].indexOf(ext.name) < 0).concat(Document), + content: { type: 'doc', content: [] }, + }, + [] + ); const startPowerpoint = useCallback(() => { toggleVisible(true); diff --git a/packages/client/src/components/document/link/index.tsx b/packages/client/src/components/document/link/index.tsx index 53a3fc8d..928e3122 100644 --- a/packages/client/src/components/document/link/index.tsx +++ b/packages/client/src/components/document/link/index.tsx @@ -14,6 +14,8 @@ interface IProps { const { Text } = Typography; +const style = { cursor: 'pointer' }; + export const DocumentLinkCopyer: React.FC = ({ organizationId, wikiId, documentId, render }) => { const handle = useCallback(() => { copy(buildUrl(`/app/org/${organizationId}/wiki/${wikiId}/doc/${documentId}`)); @@ -29,7 +31,7 @@ export const DocumentLinkCopyer: React.FC = ({ organizationId, wikiId, d return render ? ( <>{render({ copy: handle, children: content })} ) : ( - + {content} ); diff --git a/packages/client/src/components/document/reader/author.tsx b/packages/client/src/components/document/reader/author.tsx index 405e52c9..ec44d294 100644 --- a/packages/client/src/components/document/reader/author.tsx +++ b/packages/client/src/components/document/reader/author.tsx @@ -8,18 +8,18 @@ interface IProps { document: IDocument; } +const style = { + borderTop: '1px solid var(--semi-color-border)', + marginTop: '0.75em', + padding: '16px 0', + fontSize: 13, + fontWeight: 'normal', + color: 'var(--semi-color-text-0)', +}; + export const Author: React.FC = ({ document }) => { return ( -
+
diff --git a/packages/client/src/components/document/reader/index.tsx b/packages/client/src/components/document/reader/index.tsx index c2c12d09..68ce6cff 100644 --- a/packages/client/src/components/document/reader/index.tsx +++ b/packages/client/src/components/document/reader/index.tsx @@ -29,6 +29,14 @@ interface IProps { documentId: string; } +const loadingStyle = { + minHeight: 240, + display: 'flex', + alignItems: 'center', + justifyContent: 'center', + margin: 'auto', +}; + export const DocumentReader: React.FC = ({ documentId }) => { const { isMobile } = IsOnMobile.useHook(); const mounted = useMount(); @@ -132,15 +140,7 @@ export const DocumentReader: React.FC = ({ documentId }) => { +
} diff --git a/packages/client/src/components/document/reader/public/index.tsx b/packages/client/src/components/document/reader/public/index.tsx index ace592d0..945aa199 100644 --- a/packages/client/src/components/document/reader/public/index.tsx +++ b/packages/client/src/components/document/reader/public/index.tsx @@ -9,7 +9,6 @@ import { Seo } from 'components/seo'; import { Theme } from 'components/theme'; import { User } from 'components/user'; import { usePublicDocumentDetail } from 'data/document'; -import { useDocumentStyle } from 'hooks/use-document-style'; import { useMount } from 'hooks/use-mount'; import { IsOnMobile } from 'hooks/use-on-mobile'; import { SecureDocumentIllustration } from 'illustrations/secure-document'; @@ -38,12 +37,8 @@ export const DocumentPublicReader: React.FC = ({ documentId, hideLogo = const mounted = useMount(); const { wikiId: currentWikiId } = useRouterQuery<{ wikiId: IWiki['id']; documentId: IDocument['id'] }>(); const { data, loading, error, query } = usePublicDocumentDetail(documentId); - const { width, fontSize } = useDocumentStyle(); const { isMobile } = IsOnMobile.useHook(); - const editorWrapClassNames = useMemo(() => { - return width === 'standardWidth' ? styles.isStandardWidth : styles.isFullWidth; - }, [width]); - + const renderAuthor = useCallback( (element) => { if (!document) return null; diff --git a/packages/client/src/components/document/star/index.tsx b/packages/client/src/components/document/star/index.tsx index 0ec9da31..0583ead8 100644 --- a/packages/client/src/components/document/star/index.tsx +++ b/packages/client/src/components/document/star/index.tsx @@ -33,6 +33,15 @@ export const DocumentStar: React.FC = ({ organizationId, wikiId, documen [toggleVisible] ); + const toggleStarAction = useCallback( + (e) => { + e.stopPropagation(); + e.preventDefault(); + toggleStar(); + }, + [toggleStar] + ); + return ( {render ? ( @@ -46,11 +55,7 @@ export const DocumentStar: React.FC = ({ organizationId, wikiId, documen color: data ? 'rgba(var(--semi-amber-4), 1)' : 'rgba(var(--semi-grey-3), 1)', }} disabled={disabled} - onClick={(e) => { - e.stopPropagation(); - e.preventDefault(); - toggleStar(); - }} + onClick={toggleStarAction} /> )} diff --git a/packages/client/src/components/document/version/index.tsx b/packages/client/src/components/document/version/index.tsx index 738f0154..fff5b593 100644 --- a/packages/client/src/components/document/version/index.tsx +++ b/packages/client/src/components/document/version/index.tsx @@ -31,16 +31,19 @@ export const DocumentVersion: React.FC> = ({ documentId, onSelec const [selectedVersion, setSelectedVersion] = useState(null); const [diffVersion, setDiffVersion] = useState(null); - const editor = useEditor({ - editable: false, - editorProps: { - attributes: { - class: 'is-editable', + const editor = useEditor( + { + editable: false, + editorProps: { + attributes: { + class: 'is-editable', + }, }, + extensions: CollaborationKit, + content: { type: 'doc', content: [] }, }, - extensions: CollaborationKit, - content: { type: 'doc', content: [] }, - }); + [] + ); const close = useCallback(() => { toggleVisible(false); diff --git a/packages/client/src/components/wiki/setting/import/index.tsx b/packages/client/src/components/wiki/setting/import/index.tsx index eeb81188..fe93dae7 100644 --- a/packages/client/src/components/wiki/setting/import/index.tsx +++ b/packages/client/src/components/wiki/setting/import/index.tsx @@ -94,7 +94,7 @@ export const Import: React.FC = ({ wikiId }) => {
+ `/app/org/${document.organizationId}/wiki/${document.wikiId}/doc/${document.id}`; + export const WikiTocs: React.FC = ({ wikiId, docAsLink = '/app/org/[organizationId]/wiki/[wikiId]/doc/[documentId]', - getDocLink = (document) => `/app/org/${document.organizationId}/wiki/${document.wikiId}/doc/${document.id}`, + getDocLink = defaultGetDocLink, }) => { const { pathname, query } = useRouter(); const { data: wiki, loading: wikiLoading, error: wikiError } = useWikiDetail(wikiId); @@ -129,7 +132,9 @@ export const WikiTocs: React.FC = ({ {wiki.name} - + + +
) : ( diff --git a/packages/client/src/components/wiki/tocs/manager/index.tsx b/packages/client/src/components/wiki/tocs/manager/index.tsx index 1ae3fbfe..e388b03e 100644 --- a/packages/client/src/components/wiki/tocs/manager/index.tsx +++ b/packages/client/src/components/wiki/tocs/manager/index.tsx @@ -1,6 +1,5 @@ import { Banner, Button, Toast, Tree, Typography } from '@douyinfe/semi-ui'; import { DataRender } from 'components/data-render'; -import { Resizeable } from 'components/resizeable'; import { useWikiTocs } from 'data/wiki'; import React, { useCallback, useEffect, useState } from 'react'; @@ -18,7 +17,7 @@ interface IDataNode { children?: Array; } -const { Title, Text } = Typography; +const { Text } = Typography; const extractRelation = (treeData: Array) => { const res = []; @@ -41,6 +40,8 @@ const extractRelation = (treeData: Array) => { return res; }; +const marginBottomStyle = { marginBottom: 16 }; + export const WikiTocsManager: React.FC = ({ wikiId }) => { const { data: tocs, loading: tocsLoading, error: tocsError, update: updateTocs } = useWikiTocs(wikiId); @@ -105,6 +106,10 @@ export const WikiTocsManager: React.FC = ({ wikiId }) => { [treeData] ); + const renderNorContent = useCallback(() => { + return ; + }, [treeData, onDrop]); + const submit = useCallback(() => { const data = extractRelation(treeData); updateTocs(data).then(() => { @@ -121,16 +126,10 @@ export const WikiTocsManager: React.FC = ({ wikiId }) => { icon={null} closeIcon={null} description={在下方进行拖拽以重新整理目录结构} - style={{ marginBottom: 16 }} + style={marginBottomStyle} />
- { - return ; - }} - /> +
diff --git a/packages/client/src/data/organization.ts b/packages/client/src/data/organization.ts index 009cf229..5b6edc54 100644 --- a/packages/client/src/data/organization.ts +++ b/packages/client/src/data/organization.ts @@ -11,7 +11,7 @@ import { HttpClient } from 'services/http-client'; */ export const useCreateOrganization = () => { const [apiWithLoading, loading] = useAsyncLoading((data) => - HttpClient.request({ + HttpClient.request({ method: OrganizationApiDefinition.createOrganization.method, url: OrganizationApiDefinition.createOrganization.client(), data, diff --git a/packages/client/src/data/wiki.ts b/packages/client/src/data/wiki.ts index 86ef1bf3..213a8329 100644 --- a/packages/client/src/data/wiki.ts +++ b/packages/client/src/data/wiki.ts @@ -319,7 +319,7 @@ export const useWikiDocuments = (wikiId) => { export const getWikiMembers = ( wikiId, page, - pageSize, + pageSize = 12, cookie = null ): Promise<{ data: Array<{ auth: IAuth; user: IUser }>; total: number }> => { return HttpClient.request({ diff --git a/packages/client/src/tiptap/editor/collaboration/collaboration/index.tsx b/packages/client/src/tiptap/editor/collaboration/collaboration/index.tsx index 7f428af8..9abaad84 100644 --- a/packages/client/src/tiptap/editor/collaboration/collaboration/index.tsx +++ b/packages/client/src/tiptap/editor/collaboration/collaboration/index.tsx @@ -4,7 +4,7 @@ import { DataRender } from 'components/data-render'; import deepEqual from 'deep-equal'; import { useToggle } from 'hooks/use-toggle'; import { SecureDocumentIllustration } from 'illustrations/secure-document'; -import React, { forwardRef, useEffect, useImperativeHandle, useMemo, useRef, useState } from 'react'; +import React, { forwardRef, useCallback, useEffect, useImperativeHandle, useMemo, useRef, useState } from 'react'; import { Editor } from 'tiptap/core'; import { IndexeddbPersistence } from 'tiptap/core/thritypart/y-indexeddb'; @@ -18,6 +18,14 @@ export type ICollaborationRefProps = { getEditor: () => Editor; }; +const errorContainerStyle = { + margin: '10%', + display: 'flex', + justifyContent: 'center', + flexDirection: 'column', + alignItems: 'center', +} as React.CSSProperties; + export const CollaborationEditor = forwardRef((props: ICollaborationEditorProps, ref) => { const { id: documentId, @@ -56,9 +64,9 @@ export const CollaborationEditor = forwardRef((props: ICollaborationEditorProps, onAwarenessUpdate && onAwarenessUpdate(users); lastAwarenessRef.current = users; }, - onAuthenticationFailed() { + onAuthenticationFailed(e) { toggleLoading(false); - setError(new Error('鉴权失败!暂时无法提供服务')); + setError(e || new Error('鉴权失败!暂时无法提供服务')); }, onSynced() { toggleLoading(false); @@ -69,6 +77,34 @@ export const CollaborationEditor = forwardRef((props: ICollaborationEditorProps, } as any); }, [documentId, user, type, editable, onAwarenessUpdate, toggleLoading]); + const renderEditor = useCallback( + () => ( + + ), + [documentId, editable, hideComment, hocuspocusProvider, menubar, onTitleUpdate, renderInEditorPortal, status, user] + ); + + const renderError = useCallback( + (error) => ( +
+ + {(error && error.message) || '未知错误'} +
+ ), + [] + ); + useImperativeHandle( ref, () => @@ -101,49 +137,19 @@ export const CollaborationEditor = forwardRef((props: ICollaborationEditorProps, }, [hocuspocusProvider]); return ( - <> -
- - -
- } - error={error} - errorContent={(error) => ( -
- - - {(error && error.message) || '未知错误'} - -
- )} - normalContent={() => ( - - )} - /> -
- +
+ + +
+ } + error={error} + errorContent={renderError} + normalContent={renderEditor} + /> +
); }); diff --git a/packages/server/src/dtos/login-user.dto.ts b/packages/server/src/dtos/login-user.dto.ts index a1e1da1e..8338ea96 100644 --- a/packages/server/src/dtos/login-user.dto.ts +++ b/packages/server/src/dtos/login-user.dto.ts @@ -3,7 +3,7 @@ import { IsNotEmpty, IsString, MinLength } from 'class-validator'; export class LoginUserDto { @IsString({ message: '用户名称类型错误(正确类型为:String)' }) @IsNotEmpty({ message: '用户账号不能为空' }) - @MinLength(5, { message: '用户账号至少5个字符' }) + @MinLength(1, { message: '用户账号至少1个字符' }) readonly name: string; @IsString({ message: '用户密码类型错误(正确类型为:String)' }) diff --git a/packages/server/src/services/user.service.ts b/packages/server/src/services/user.service.ts index b97ce86a..0e602520 100644 --- a/packages/server/src/services/user.service.ts +++ b/packages/server/src/services/user.service.ts @@ -253,7 +253,7 @@ export class UserService { const currentSystemConfig = await this.systemService.getConfigFromDatabase(); const oldData = await this.userRepo.findOne(user.id); - if (oldData.email !== dto.email) { + if (oldData && dto && oldData.email !== dto.email) { if (await this.userRepo.findOne({ where: { email: dto.email } })) { throw new HttpException('该邮箱已被注册', HttpStatus.BAD_REQUEST); }