Skip to content

Conversation

@julceslauhub
Copy link
Contributor

@julceslauhub julceslauhub commented Jan 14, 2026

Issue & Reproduction Steps

Expected behavior:
Redirect to the next task that the user is assigned to

Actual behavior:
Keeps showing the interstitial page and attempts to redirect to another
task the user does not has access to

Solution

Add a condition in order to execute the redirect task only if the current user is the same that returned in the channel

How to Test

  • Import the attached process and configure users
  • Start a case as a user who has at least one assigned task after the first one
  • Submit the form
  • Wait for the redirection to the next task

Related Tickets & Packages

https://processmaker.atlassian.net/browse/FOUR-28401

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.

ci:deploy


Note

Strengthens redirect handling and stabilizes tests.

  • Task redirect logic: In src/components/task.vue, handleRedirectToTask now proceeds only when the event’s userId matches the current task’s user.id, or when task.elementDestination.type is taskSource. Preserves interstitial behavior and forces hard redirect for ConversationalForm when needed.
  • E2E updates: Refactors Cypress selectors to use stable id-prefix patterns for radio/checkbox/select options, reducing index-based flakiness (updates in ComplexScreen.spec.js, SelectListWatcher.spec.js, SingleSelectWithInvalidValue.spec.js). Adds user.id to mocked task responses and adjusts assertions in Task.spec.js to validate the new redirect constraints.

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

cursor[bot]

This comment was marked as outdated.

@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 14, 2026
@ProcessMaker ProcessMaker deleted a comment from cypress bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cypress bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cursor bot Jan 15, 2026
@ProcessMaker ProcessMaker deleted a comment from cypress bot Jan 16, 2026
@ProcessMaker ProcessMaker deleted a comment from processmaker-sonarqube bot Jan 16, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@ProcessMaker ProcessMaker deleted a comment from cypress bot Jan 16, 2026
@ProcessMaker ProcessMaker deleted a comment from cypress bot Jan 16, 2026
@cypress
Copy link

cypress bot commented Jan 16, 2026

screen-builder    Run #2184

Run Properties:  status check failed Failed #2184  •  git commit 8b9c9f94b9: FOUR-28401 Issue Identified with Parallel Task Redirect Behavior
Project screen-builder
Branch Review FOUR-28401
Run status status check failed Failed #2184
Run duration 09m 49s
Commit git commit 8b9c9f94b9: FOUR-28401 Issue Identified with Parallel Task Redirect Behavior
Committer julceslauhub
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 19
Tests that did not run due to a failure in a mocha hook  Skipped 8
Tests that passed  Passing 380
View all changes introduced in this branch ↗︎

Tests for review

Failed  tests/e2e/specs/FormSelectList.spec.js • 1 failed test • CI - Chrome

View Output

Test Artifacts
Form Select List > Validate duplicated option Test Replay Screenshots

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-143eba321e.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 f5e515d into develop Jan 19, 2026
51 of 69 checks passed
@nolanpro nolanpro deleted the FOUR-28401 branch January 19, 2026 17:27
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.

6 participants