Skip to content

Commit f0fdda0

Browse files
committed
fix(posthog): eliminate all remaining phantom empty-string workspace groups
1 parent efec6d9 commit f0fdda0

3 files changed

Lines changed: 5 additions & 3 deletions

File tree

apps/sim/app/api/workflows/[id]/deploy/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ export async function POST(request: NextRequest, { params }: { params: Promise<{
102102
'workflow_deployed',
103103
{ workflow_id: id, workspace_id: workflowData!.workspaceId ?? '' },
104104
{
105-
groups: { workspace: workflowData!.workspaceId ?? '' },
105+
groups: workflowData!.workspaceId ? { workspace: workflowData!.workspaceId } : undefined,
106106
setOnce: { first_workflow_deployed_at: new Date().toISOString() },
107107
}
108108
)

apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,9 @@ export async function POST(
113113
workspace_id: workflowRecord?.workspaceId ?? '',
114114
version,
115115
},
116-
{ groups: { workspace: workflowRecord?.workspaceId ?? '' } }
116+
workflowRecord?.workspaceId
117+
? { groups: { workspace: workflowRecord.workspaceId } }
118+
: undefined
117119
)
118120

119121
recordAudit({

apps/sim/app/api/workflows/[id]/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,7 +230,7 @@ export async function DELETE(
230230
userId,
231231
'workflow_deleted',
232232
{ workflow_id: workflowId, workspace_id: workflowData.workspaceId ?? '' },
233-
{ groups: { workspace: workflowData.workspaceId ?? '' } }
233+
workflowData.workspaceId ? { groups: { workspace: workflowData.workspaceId } } : undefined
234234
)
235235

236236
const elapsed = Date.now() - startTime

0 commit comments

Comments
 (0)