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

[Perfomance and usability] Refactor Loading #455

Open
3 of 12 tasks
antoineatstariongroup opened this issue Sep 18, 2023 · 1 comment
Open
3 of 12 tasks

[Perfomance and usability] Refactor Loading #455

antoineatstariongroup opened this issue Sep 18, 2023 · 1 comment
Labels

Comments

@antoineatstariongroup
Copy link
Contributor

antoineatstariongroup commented Sep 18, 2023

Prerequisites

  • I have written a descriptive issue title
  • I have verified that I am running the latest version of the COMET-WEB
  • I have searched open and closed issues to ensure it has not already been reported

Description

When the auto-refresh is enable with a low refresh-rate, the webapp start to be unusable.

We have to think about refactor the loading strategy and to make sure to only display the loading component when it make sense.
Moreover, we could use two new components from DevExpress : DxLoadingPanel and DxWaitIndicator

  • Remove the use of the LoadingComponent
  • Use DevExpress component inside visual component to let a panel/table/diagram display the loading by itself. It will remove the loading component for the whole page and let user use other component in the meantime
  • Make sure that every VM display the loading component when needed => Do not refresh data all the time, make sure to filter and to subscribe only when it's necessary.
  • When updating data, do not clear the whole collection and recreate everything : please recycle your row, update properties, delete what really needs to be deleted and add new entries.

Steps to Reproduce

Logs

  • There are no logs regarding the issue
  • I have attached a log file or included the relevant section in this issue

System Configuration

  • COMET version:
    • COMET-WEB:
    • Other:
  • Web Services:
    • COMET Web Services version:
@samatstariongroup samatstariongroup moved this to To do in COMET-WEB Jul 17, 2024
@joao4all joao4all self-assigned this Aug 2, 2024
@joao4all joao4all moved this from To do to In progress in COMET-WEB Aug 2, 2024
@joao4all
Copy link
Contributor

joao4all commented Aug 2, 2024

For this tasks, some work was done in this branch

@joao4all joao4all moved this from In progress to To do in COMET-WEB Aug 2, 2024
@joao4all joao4all removed their assignment Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: To do
Development

No branches or pull requests

2 participants