Process tasks: complete via Inbox endpoint, show current step, refresh on select#6075
Open
delchev wants to merge 2 commits into
Open
Process tasks: complete via Inbox endpoint, show current step, refresh on select#6075delchev wants to merge 2 commits into
delchev wants to merge 2 commits into
Conversation
…resh on select
Follow-ups on the in-context BPM task surfacing:
- FormIntentGenerator: the generated task form now completes via the per-task
permission-checked /services/inbox/tasks/{id} instead of the role-guarded
/services/bpm/bpm-processes/tasks/{id} (which blocked candidate-group users
who can claim/act but lack ADMINISTRATOR/DEVELOPER/OPERATOR).
- process-tasks directive: shows the current step (the single actionable task's
name) inline, collapsing to a count when several.
- master-list/master-manage main-details: refresh task buckets on entitySelected
so the badge stays current while navigating master records.
- IntentEngineIT.assertForm: assert the inbox endpoint + both close calls and
forbid the role-guarded BPM endpoint.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ing) Cross-reference in the root + engine-intent CLAUDE.md that the trigger-written ProcessId is read by the generated entity-view UI (shared ProcessTasks module + <entity-process-tasks> directive): how it correlates (entity.ProcessId === task.processInstanceId), which views are wired (gated on hasProcess), and that the task form completes via the permission-checked Inbox endpoint and self-closes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-ups on the in-context BPM user-task surfacing merged in #6074.
Changes
COMPLETEto the per-task permission-checked/services/inbox/tasks/{id}instead of the role-guarded/services/bpm/bpm-processes/tasks/{id}. The latter requiresADMINISTRATOR/DEVELOPER/OPERATOR, so a candidate-group user who can claim and act on a task could still be refused on completion. (FormIntentGenerator)<entity-process-tasks>control now labels itself with the single actionable task's name (answers "why is there a task here?" at a glance), collapsing to a count when several. (process-tasks.js)master-list/master-managemain-detailspanes, task buckets are refreshed onentitySelected, so the badge stays current while navigating master records.IntentEngineIT.assertFormnow asserts the Inbox endpoint and both close calls (closeWindow()+window.close()), and forbids the role-guarded BPM endpoint — locking in the completion-path and close behaviour. (HTTP-only, no Selenide.)Notes
The browser-side directive (
process-tasks.js) and template wiring remain exercised manually; a full Selenide IT would need the intent sample repo to carry a process-triggered entity + form (a cross-repo effort), so it's deferred.🤖 Generated with Claude Code