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.

Sign up for GitHub

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

Jump to bottom

Improve workflow instance details display speed & Blazor monaco feedback #72

Closed
JBBianchi opened this issue May 9, 2022 · 0 comments · Fixed by #78
Closed

Improve workflow instance details display speed & Blazor monaco feedback #72

JBBianchi opened this issue May 9, 2022 · 0 comments · Fixed by #78
Assignees
@JBBianchi
Labels
app: dashboard Concerns the Dashboard UI priority: low Indicates a low priority issue type: enhancement New feature or request weight: 1 An issue that has a very low development impact
Milestone
v0.2.0

Comments

@JBBianchi
Copy link
Member

What would you like to be added:
Checking the details of a workflow instance can be very slow, the more activities (input/output payloads), the slower it gets. This is due to Blazor Monaco being a little slow and the accumulation its of instances in the workflow instance details. To prevent that problem, accordions should not render their body if it's not "expanded" (instead of just toggling css classes).
It would also be nice to add a loader indicator while Blazor Monaco is processing the data.

Why is this needed:
Better UX.
@JBBianchi JBBianchi added type: enhancement New feature or request app: dashboard Concerns the Dashboard UI priority: low Indicates a low priority issue weight: 1 An issue that has a very low development impact labels May 9, 2022
@JBBianchi JBBianchi added this to the 0.2.0 milestone May 9, 2022
@JBBianchi JBBianchi self-assigned this May 9, 2022
JBBianchi added a commit to neuroglia-io/synapse that referenced this issue May 9, 2022
@JBBianchi
fix(Dashboard): Improved instance details display reactivity
e1f57e1 
Closes serverlessworkflow#72

Signed-off-by: JBBianchi <jb.bianchi@neuroglia.io>
JBBianchi added a commit to neuroglia-io/synapse that referenced this issue May 10, 2022
@JBBianchi
fix(Dashboard): Improved instance details display reactivity
086cdf0 
Closes serverlessworkflow#72

Signed-off-by: JBBianchi <jb.bianchi@neuroglia.io>
@JBBianchi JBBianchi mentioned this issue May 10, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JBBianchi JBBianchi

Labels
app: dashboard Concerns the Dashboard UI priority: low Indicates a low priority issue type: enhancement New feature or request weight: 1 An issue that has a very low development impact
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

fix(Dashboard): Improved instance details display reactivity neuroglia-io/synapse
1 participant
@JBBianchi