Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix how we handle workflow failures #1762

Closed
fazil-ey opened this issue May 7, 2024 · 8 comments
Closed

Fix how we handle workflow failures #1762

fazil-ey opened this issue May 7, 2024 · 8 comments
Assignees
Labels
Application General feature to be available globally across the platform Explore Investigation by BUS and/or DEV Maintenance

Comments

@fazil-ey
Copy link
Collaborator

fazil-ey commented May 7, 2024

if there is a workflow failure, The status of the form is updates to APPROVED in submitted forms but it remains in the tasklist of the supervisor as NEW. This can potentially cause confusion where an employee thinks their form is submitted for documentation or leave application is submitted to AskMyHR but in reality it is not.

Video - https://bcgov.sharepoint.com/:v:/r/teams/03991/Shared%20Documents/Conflict%20of%20Interest/Test%20recordings%20and%20screenshots/Screen%20Recording%202024-05-06%20at%203.48.41%20PM.mov?csf=1&web=1&e=VOvE1I

Explore potential solutions -

  1. show error message to supervisor if the CRM ticket is not created successfully.
  2. if there is an error do not show "APPROVED" to employee
@fazil-ey fazil-ey added Conflict of Interest (CoI) Tickets related to the CoI Disclosure form 🐛 Bug Something isn't working labels May 7, 2024
@bhumin-fw
Copy link
Collaborator

Once form successfully approved, That will be removed from "review and process" tab. I have requested access to review above video. Mean while you can review this again and share your comment on it.

@fazil-ey
Copy link
Collaborator Author

fazil-ey commented May 9, 2024

Similar to telework, since after submission by supervisor edits are not allowed, the task will be removed from the Review and Process tab, therefore approved forms will not show there. The forms we were seeing with the new status were forms where the workflow failed.

@fazil-ey fazil-ey added Explore Investigation by BUS and/or DEV Application General feature to be available globally across the platform and removed 🐛 Bug Something isn't working labels May 9, 2024
@fazil-ey fazil-ey changed the title COI: Status not updating in Review and Process tab Fix how we handle workflow failures May 9, 2024
@jerry-ey jerry-ey self-assigned this May 9, 2024
@fazil-ey fazil-ey removed the Conflict of Interest (CoI) Tickets related to the CoI Disclosure form label May 10, 2024
@jerry-ey
Copy link
Collaborator

jerry-ey commented May 16, 2024

Discussed with Bhumin and got the info I need. will work on the exploration after coming back from vacation.
Note:

  • can use COI wrong employee id to test.
  • need to handle error on frontend when camunda send back error message

@jerry-ey
Copy link
Collaborator

jerry-ey commented Jun 20, 2024

  • For frontend, we have idea how to implement "close loader" and "show error message".

  • For backend, Bhumin and I still investigate the part how application status is getting updated to "APPROVED"

@jerry-ey
Copy link
Collaborator

jerry-ey commented Jun 20, 2024

Draft PR for frontend: #1836
(Closing the loader and showing error toast message)

@jerry-ey jerry-ey removed their assignment Jun 21, 2024
@jerry-ey
Copy link
Collaborator

jerry-ey commented Jun 21, 2024

In order to use the remaining time effectively, I have unassigned myself and discussed handing over the task to Bhumin.

@bhumin-fw
Copy link
Collaborator

A separate ticket has been created to revert the Form.io status and data. #1874

We will display an error message to the user if a timeout error occurs as an initial step.

@fazil-ey
Copy link
Collaborator Author

fazil-ey commented Sep 5, 2024

CLosing this because the action for this will be done in #1874

@fazil-ey fazil-ey closed this as completed Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Application General feature to be available globally across the platform Explore Investigation by BUS and/or DEV Maintenance
Projects
None yet
Development

No branches or pull requests

4 participants