fix(whiteboard): don't set state when not writable (#1607) (#1616)

- errorTips: improve error log

Co-authored-by: hyrious <hyrious@outlook.com>

Author: BlackHole1

desktop/renderer-app/src/components/Tips/ErrorTips.tsx

@@ -1,10 +1,9 @@
 import { message } from "antd";
 import { ServerRequestError } from "../../utils/error/server-request-error";
-import { NODE_ENV } from "../../constants/process";
 import { i18n } from "../../utils/i18n";
 
 export const errorTips = (e: unknown): void => {
-    if (NODE_ENV === "development") {
+    if (process.env.NODE_ENV === "development") {
         console.error(e);
     }
 

desktop/renderer-app/src/stores/whiteboard-store.ts

@@ -391,6 +391,9 @@ export class WhiteboardStore {
         if (!room || !windowManager) {
             return;
         }
+        if (!room.isWritable) {
+            return;
+        }
         if (!room.state.globalState || !(room.state.globalState as any).scrollToTop) {
             room.setGlobalState({ scrollToTop: true });
             windowManager.moveCamera({ centerY: -950, animationMode: AnimationMode.Immediately });

web/flat-web/src/components/Tips/ErrorTips.tsx

@@ -1,10 +1,9 @@
 import { message } from "antd";
 import { ServerRequestError } from "../../utils/error/server-request-error";
-import { NODE_ENV } from "../../constants/process";
 import { i18n } from "../../utils/i18n";
 
 export const errorTips = (e: unknown): void => {
-    if (NODE_ENV === "development") {
+    if (process.env.NODE_ENV === "development") {
         console.error(e);
     }
 

web/flat-web/src/stores/whiteboard-store.ts

@@ -359,6 +359,9 @@ export class WhiteboardStore {
         if (!room || !windowManager) {
             return;
         }
+        if (!room.isWritable) {
+            return;
+        }
         if (!room.state.globalState || !(room.state.globalState as any).scrollToTop) {
             room.setGlobalState({ scrollToTop: true });
             windowManager.moveCamera({ centerY: -950, animationMode: AnimationMode.Immediately });