Skip to content

[improvement][UI][2.0.9] Speed up the refresh of task status in process instance #13587

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
Merged
brave-lee merged 2 commits into from
Feb 22, 2023
Merged

[improvement][UI][2.0.9] Speed up the refresh of task status in process instance #13587

brave-lee merged 2 commits into from
Feb 22, 2023

Conversation

@reele
Copy link
Contributor

@reele reele commented Feb 20, 2023

Purpose of the pull request

This PR Fix #13581.

Brief change log

-Add incremental comparison after the first refresh
-Set the auto refresh timer after the first refresh

Verify this pull request

@reele reele changed the title [Improvement][UI][2.0.x] Speed up the refresh of task status in process instance [Improvement][UI][2.0.9-prepare] Speed up the refresh of task status in process instance Feb 20, 2023
@reele reele changed the title [Improvement][UI][2.0.9-prepare] Speed up the refresh of task status in process instance [Improvement][UI][2.0.9] Speed up the refresh of task status in process instance Feb 20, 2023
@reele reele changed the title [Improvement][UI][2.0.9] Speed up the refresh of task status in process instance [improvement][UI][2.0.9] Speed up the refresh of task status in process instance Feb 20, 2023
@EricGao888 EricGao888 added this to the 2.0.9 milestone Feb 20, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 20, 2023

Codecov Report

❗ No coverage uploaded for pull request base (2.0.9-prepare@a10483f). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 1f50846 differs from pull request most recent head d008708. Consider uploading reports for the commit d008708 to get more accurate results

@@               Coverage Diff                @@
##             2.0.9-prepare   #13587   +/-   ##
================================================
  Coverage                 ?   20.64%           
  Complexity               ?     1603           
================================================
  Files                    ?      540           
  Lines                    ?    23110           
  Branches                 ?     2771           
================================================
  Hits                     ?     4772           
  Misses                   ?    17845           
  Partials                 ?      493

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@EricGao888 EricGao888 added the first time contributor First-time contributor label Feb 21, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 22, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug E 62 Bugs
Vulnerability E 13 Vulnerabilities
Security Hotspot E 12 Security Hotspots
Code Smell A 1088 Code Smells

17.0% 17.0% Coverage
5.4% 5.4% Duplication

brave-lee
brave-lee approved these changes Feb 22, 2023
View reviewed changes
Copy link
Contributor

@brave-lee brave-lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@brave-lee brave-lee merged commit 33e345f into apache:2.0.9-prepare Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brave-lee brave-lee brave-lee approved these changes

@Amy0104 Amy0104 Awaiting requested review from Amy0104

@songjianet songjianet Awaiting requested review from songjianet

Assignees

@reele reele

Labels

first time contributor First-time contributor

Projects

None yet

Milestone

2.0.9

Development

Successfully merging this pull request may close these issues.

4 participants

@reele @codecov-commenter @brave-lee @EricGao888