Skip to content

[Fix][UI Next][V1.0.0-Alpha] Fix the state column of task instance ta…#8987

Merged
songjianet merged 3 commits into
apache:devfrom
Amy0104:fix-8982
Mar 18, 2022
Merged

[Fix][UI Next][V1.0.0-Alpha] Fix the state column of task instance ta…#8987
songjianet merged 3 commits into
apache:devfrom
Amy0104:fix-8982

Conversation

@Amy0104
Copy link
Copy Markdown
Member

@Amy0104 Amy0104 commented Mar 18, 2022

Purpose of the pull request

This pull request fixs the state column of task instance table displays error.

Releated issues

close #8982

@songjianet songjianet added bug Something isn't working UI ui and front end related labels Mar 18, 2022
@songjianet
Copy link
Copy Markdown
Member

image

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #8987 (8424eb8) into dev (045ef75) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8987      +/-   ##
============================================
- Coverage     39.77%   39.76%   -0.01%     
+ Complexity     4265     4264       -1     
============================================
  Files           808      808              
  Lines         32550    32550              
  Branches       3649     3649              
============================================
- Hits          12946    12945       -1     
  Misses        18378    18378              
- Partials       1226     1227       +1     
Impacted Files Coverage Δ
...dolphinscheduler/remote/future/ResponseFuture.java 81.96% <0.00%> (-1.64%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 045ef75...8424eb8. Read the comment docs.

{
size: '18px',
style: 'position: relative; top: 7.5px; left: 7.5px',
class: stateIcon.classNames
Copy link
Copy Markdown
Member

@SbloodyS SbloodyS Mar 18, 2022

Choose a reason for hiding this comment

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

The WorkflowE2ETest using class success to verify the process instance success state. Removing it would lead to the WorkflowE2ETest failed.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I will add the class by the task type.Such as SUBMITTED_SUCCESS will be submitted_success expect FAILURE which corresponds to failed.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Copy Markdown
Member

@songjianet songjianet left a comment

Choose a reason for hiding this comment

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

LGTM

@songjianet songjianet merged commit 85b2352 into apache:dev Mar 18, 2022
@Amy0104 Amy0104 deleted the fix-8982 branch April 12, 2022 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][UI Next][V1.0.0-Alpha] The state column of task instance table displays error.

4 participants