Skip to content

Conversation

@marcoAntonioNina
Copy link
Contributor

@marcoAntonioNina marcoAntonioNina commented Sep 30, 2025

Issue & Reproduction Steps

It is necessary to allow more than one manager to be added to the processes, with a limit of only adding 10 managers.

Solution

  • Change in section configuration in Process

How to Test

Check the process configuration where you can add managers and see that you can only add a maximum of 10.
These changes need to be tested in the views where the manager has access.

We need to verify that a process's managers can still view cases.
We also need to verify that process imports and exports are free of issues and that they continue to work for users who were dependent on the process.

We need to verify that the manager assignment is correct.
We need to review the process creation and process configuration interfaces to ensure that manager assignments are correct.
We need to verify notifications to managers.

managers.mov

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

Note

Adds multi-manager support (up to 10) with true round-robin assignments, updates APIs, UI, export/import, notifications, and access checks accordingly.

  • Process model and assignment:
    • properties->manager_id now array; added getManagers()/setManagers() and updated manager() accessor.
    • True round-robin manager selection across requests in ProcessManagerAssigned; fallback logic added in FormalExpression.
    • Process::getNextUser/checkAssignment updated to use new rule and token signature.
  • API/controllers:
    • ProcessController: accepts/validates multiple manager_id (max 10), serializes arrays; start-event checks use array membership.
    • Cases/tasks views and channels auth updated to check in_array(user, manager_id).
  • Policies/permissions:
    • ProcessPolicy, ProcessVersionPolicy, ProcessRequestTokenPolicy updated for multi-manager checks and rollback/reassign rules.
    • Task queries (TaskControllerIndexMethods) support JSON array manager filtering.
  • Export/Import:
    • Exporter now outputs process_manager (names array) and process_manager_id (IDs array); ProcessExporter handles multiple manager dependents and imports arrays.
    • UI export summary displays list of managers.
  • Frontend (Vue):
    • SelectUser supports multi-select with selection info and max limit.
    • Process create/edit modals accept multiple managers and send JSON array.
  • Notifications:
    • Error notifications sent to all managers.
  • Docs/i18n/tests:
    • New i18n strings for limits/selected; tests adjusted for array-based managers.

Written by Cursor Bugbot for commit 9d5adef. This will update automatically on new commits. Configure here.

ci:deploy
ci:package-advanced-user-manager:task/FOUR-26777

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-48283ef2f2.engk8s.processmaker.net

cursor[bot]

This comment was marked as outdated.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-48283ef2f2.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-48283ef2f2.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-48283ef2f2.engk8s.processmaker.net

@cursor
Copy link

cursor bot commented Oct 27, 2025

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on November 21.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-48283ef2f2.engk8s.processmaker.net

@processmaker-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nolanpro nolanpro merged commit 8a60016 into develop Oct 28, 2025
8 checks passed
@nolanpro nolanpro deleted the task/FOUR-26777 branch October 28, 2025 00:11
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.

5 participants