Skip to content

FOUR-26707 Implement Conditional Destination Redirect#8544

Merged
nolanpro merged 13 commits intodevelopfrom
feature/FOUR-26707
Nov 21, 2025
Merged

FOUR-26707 Implement Conditional Destination Redirect#8544
nolanpro merged 13 commits intodevelopfrom
feature/FOUR-26707

Conversation

@caleeli
Copy link
Contributor

@caleeli caleeli commented Sep 26, 2025 •
edited by cursor bot
Loading

Implement Conditional Destination Redirect

Solution

  • Implement backend to support extra conditioned redirection rules for a Task.
  • Add to the modeler Inspector the new conditioned redirection rules.
image

How to Test

  • Create a process like:
  • Setup the conditionl redirection rules
  • Note that the last task rules are overwritten by the End Event redirection

Related Tickets & Packages

FOUR-26707

ProcessMaker/modeler#1947
ProcessMaker/screen-builder#1863

ci:modeler:feature/FOUR-26707
ci:screen-builder:feature/FOUR-26707
ci:deploy

.
.
.
.

Note

Introduces a FEEL-based conditional redirect service and integrates it to resolve task destinations, updates event payloads and modeler behavior, and adds comprehensive tests.

  • Backend:
    • Service/Contract: Add ConditionalRedirectService with FEEL evaluation and ConditionalRedirectServiceInterface; bind in ProcessMakerServiceProvider.
    • Task Destination: Update ProcessRequestToken to evaluate conditionalRedirect when computing elementDestination (supports taskList, homepageDashboard, processLaunchpad, customDashboard, externalURL, displayNextAssignedTask, taskSource).
    • Events: Include elementDestination on ProcessUpdated broadcasts.
  • Frontend (Modeler):
    • Update TaskInterstitial.vue to respect conditional redirects when toggling interstitials and prevent disabling when destination is displayNextAssignedTask.
  • Tests:
    • Add feature tests for conditional redirect outcomes and unit tests for service resolution; include BPMN fixture tests/Feature/ImportExport/fixtures/conditional_redirect.xml.

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

@vladyrichter
Copy link

vladyrichter commented Sep 26, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

Miguel Angel and others added 3 commits September 29, 2025 15:44
@vladyrichter
Copy link

vladyrichter commented Sep 30, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

3 similar comments
@vladyrichter
Copy link

vladyrichter commented Oct 1, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@vladyrichter
Copy link

vladyrichter commented Oct 1, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@vladyrichter
Copy link

vladyrichter commented Oct 1, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

Miguel Angel and others added 2 commits October 3, 2025 15:17
feat: add elementDestination and conditionalRedirect computed properties
87b29b7
@caleeli
Merge pull request #8553 from ProcessMaker/FOUR-26806
2c51fe2 
FOUR-26806 User cannot select a display screen when the condition is selected Display Next Assigned Task option
@processmaker-sonarqube
Copy link

processmaker-sonarqube bot commented Oct 3, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

@vladyrichter
Copy link

vladyrichter commented Oct 3, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@vladyrichter
Copy link

vladyrichter commented Nov 7, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

1 similar comment
@vladyrichter
Copy link

vladyrichter commented Nov 10, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@vladyrichter
Copy link

vladyrichter commented Nov 10, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@caleeli
Merge pull request #8592 from ProcessMaker/FOUR-27198
ca02fac 
FOUR-27198 Request > Timeline is displaying many messages when one condition is accomplished
@vladyrichter
Copy link

vladyrichter commented Nov 11, 2025

QA server K8S was successfully deployed https://ci-031b879c58.engk8s.processmaker.net

@nolanpro nolanpro merged commit 272750c into develop Nov 21, 2025
5 of 10 checks passed
@nolanpro nolanpro deleted the feature/FOUR-26707 branch November 21, 2025 16:40
@processmaker-sonarqube
Copy link

processmaker-sonarqube bot commented Nov 21, 2025

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube

nolanpro added a commit that referenced this pull request Dec 8, 2025
@nolanpro
Revert "Merge pull request #8544 from ProcessMaker/feature/FOUR-26707"
b6a50b2 
This reverts commit 272750c, reversing
changes made to a804b1a.
nolanpro added a commit that referenced this pull request Jan 30, 2026
@nolanpro
Reapply "Merge pull request #8544 from ProcessMaker/feature/FOUR-26707"
604aa9d 
This reverts commit b6a50b2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devmiguelangel devmiguelangel devmiguelangel approved these changes

Assignees

No one assigned

Labels

4.15.11

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@caleeli @vladyrichter @devmiguelangel @nolanpro @pmPaulis