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

feat: add resource allocation interaction (1st step) #10

Merged
merged 35 commits into from
Mar 14, 2023

Conversation

assynour
Copy link
Contributor

@assynour assynour commented Mar 13, 2023

When clicking on the "Allocate more resource" button, the sub-process diagram is displayed.

  • It's running task is highlighted and the path leading to it is modified as well. This task proposed a popover which opens on hover.
  • In the future, the style of some elements of the diagram will be modified when hovering on the row of the popover.
    When clicking on the "Contact-Client" button, a window alert is displayed. In the future, it will display the 2nd process of the diagram and it will let interact with it.

Implementation notes

This PR introduces refactoring to better encapsulate the logic.
In particular, it provides classes that may be reused outside this demo and served as starting point for experimental features

The case monitoring data are fully managed by dedicated classes both for the main process and the sub-process.
Other parts of the code are still leaking the use cases at several places. This will be refactored later to limit the size of this PR.

Screenshots

Note: we see that several event listeners are registered on the button (as the window alert opens several times)

bonita-day-demo-2023_PR-10.mp4

@github-actions
Copy link

github-actions bot commented Mar 13, 2023

♻️ PR Preview c20f3ca has been successfully destroyed since this PR has been closed.

🤖 By surge-preview

@tbouffard tbouffard changed the title feat: add resource allocation interaction feat: add resource allocation interaction (1st step) Mar 14, 2023
@tbouffard tbouffard added the enhancement New feature or request label Mar 14, 2023
@tbouffard tbouffard marked this pull request as ready for review March 14, 2023 13:40
@tbouffard tbouffard merged commit bcf46a9 into main Mar 14, 2023
@tbouffard tbouffard deleted the add_resource_allocation branch March 14, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants