Skip to content

Commit 042f91f

Browse files
committed
docs(approvals): state who can see a request, now that it is participant-scoped
The drift check flagged approvals.mdx, and rightly: it documented `approverId` as one filter among several and never said who may see what — so a reader would reasonably assume omitting it returns the whole tenant, which is exactly what this PR stops being true. Also records that a decision's attachments inherit the same rule, since that is the non-obvious half: the file is as readable as the decision, never more. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SHpGw3GBA9aFpfwVArRWfd
1 parent 6644f22 commit 042f91f

1 file changed

Lines changed: 14 additions & 0 deletions

File tree

content/docs/automation/approvals.mdx

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -244,6 +244,20 @@ a person's identities in one call.
244244
Other filters: `status`, `object`, `recordId`, `submitterId`, `q`, `limit`,
245245
`offset`.
246246

247+
<Callout type="warn">
248+
**`approverId` is a filter, not authorization.** What you may see is decided
249+
separately: a request is visible to its **participants** — the submitter, a
250+
current approver, and anyone who has already acted on it (a past approver whose
251+
slot has moved on, a commenter). So omitting `approverId` returns *your*
252+
requests, not every request in the tenant. Admins with override authority
253+
(`admin_full_access`, or `organization_admin` within their org) see all of them
254+
— that is what the "all requests" view is for.
255+
256+
The same rule governs a decision's files: an attachment is exactly as readable
257+
as the decision it hangs off, never more (`sys_approval_action` delegates that
258+
question back to this service via `fileAccessDelegate`).
259+
</Callout>
260+
247261
<Callout type="warn">
248262
**Opening a request notifies nobody.** There is no built-in "you have an
249263
approval waiting" message today — an approver only discovers it by looking at

0 commit comments

Comments
 (0)