Skip to content

Bugfix/FOUR-5159: Summary screen is not displayed in sub process #4262

Merged
ryancooley merged 1 commit intodevelopfrom
bugfix/FOUR-5159
Jan 28, 2022
Merged

Bugfix/FOUR-5159: Summary screen is not displayed in sub process #4262
ryancooley merged 1 commit intodevelopfrom
bugfix/FOUR-5159

Conversation

@agustinbusso
Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

It was reported as a bug that subprocess inside a main process are not showing the summary screen in the subprocess, but that is not the correct behavior. It was fixed the issue related with the interstitial screen getting stuck and not getting
the next task.

  • Steps to Reproduce:
  • Create a process with the next configuration
  • Start Event -> Task -> Sub Process -> End Event
  • Create a sub process
  • Add Start Event -> Task(Assign Screen Interstitial) -> End Event (Assign Summary Screen)
  • Create a request
  • Route to the sub process
  • Open the sub process
  • Complete the task

Solution

  • Modified the flow to follow the next rules (For more clarify please watch the videos for the different scenarios):
  1. Summary screen in a subprocess will show only if the process is running as standalone.
  2. If a task has the option Display Next Assigned Task to Task Assignee checked, it will get the next task (if you are in a subprocess will get the next task in that subprocess, when the subprocess completed all the tasks will get the next task of the parent process)
  3. If a task has the option Display Next Assigned Task to Task Assignee unchecked, will show the task list
  4. If a subprocess is completed and no tasks in the main process, will show the summary screen of the main process

How to Test

Run e2e test cases in Task.spec.js

Please watch the following videos:

Case 1

Screen.Recording.2022-01-24.at.12.51.11.mov

Case 2

Screen.Recording.2022-01-24.at.12.56.50.mov

Case 3

Screen.Recording.2022-01-24.at.13.44.16.mov

Case 4

Screen.Recording.2022-01-24.at.16.26.59.mov

Case 5

Screen.Recording.2022-01-24.at.12.54.19.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.

@pmkerberos
Copy link
Copy Markdown

QA server was successfully deployed https://bugfix-four-5159-processmaker.ci.processmaker.net

@ryancooley ryancooley merged commit 8b62a35 into develop Jan 28, 2022
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.

4 participants