Skip to content

Check DeletePermission per job in DeleteStatusAction.getView - #8035

Open
ankurjuneja wants to merge 3 commits into
developfrom
fb_pipelineDeleteStatus
Open

Check DeletePermission per job in DeleteStatusAction.getView#8035
ankurjuneja wants to merge 3 commits into
developfrom
fb_pipelineDeleteStatus

Conversation

@ankurjuneja

@ankurjuneja ankurjuneja commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Rationale

Check DeletePermission per job in DeleteStatusAction.getView.

Related Pull Requests

Changes

  • The confirmation page now resolves jobs using the same container scope and permission checks the delete applies
  • Confirm Delete acts on exactly the jobs the page listed

…hat it displays

Report failures instead of silently skipping rows the container-filtered DELETE can't reach.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C44xjpwocskmf8ndHXRpcw
@ankurjuneja
ankurjuneja force-pushed the fb_pipelineDeleteStatus branch from 69a46bb to 8a5e13b Compare September 11, 2026 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant