fix(OUT-2914): added a check for workspace id before uploading an attachment. workspaceId is needed for building folder structure effecting filePaths

arpandhakal · arpandhakal · commit 1a1bcbbcb25b · 2026-01-20T15:08:21.000+05:45
diff --git a/src/utils/createUploadFn.ts b/src/utils/createUploadFn.ts
@@ -16,7 +16,7 @@ export const createUploadFn = (config: UploadConfig) => {
   return async (file: File) => {
     config.onUploadStart?.()
     const entityId = config.getEntityId?.() ?? null //lazily loading the entityId because some of the ids are optimistic id and we want the real ids of comments/replies
-    if (!config.token) {
+    if (!config.token || !config.workspaceId) {
       return undefined
     }
     try {

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ export const createUploadFn = (config: UploadConfig) => {`
`16`	`16`	`return async (file: File) => {`
`17`	`17`	`config.onUploadStart?.()`
`18`	`18`	`const entityId = config.getEntityId?.() ?? null //lazily loading the entityId because some of the ids are optimistic id and we want the real ids of comments/replies`
`19`		`- if (!config.token) {`
	`19`	`+ if (!config.token \|\| !config.workspaceId) {`
`20`	`20`	`return undefined`
`21`	`21`	`}`
`22`	`22`	`try {`