Skip to content

[Fix][UI Next][V1.0.0-Alpha] fix queue display bug in tenant modal - #9064

Merged
songjianet merged 2 commits into
apache:devfrom
devosend:fix/tenent-manage-queue
Mar 22, 2022
Merged

[Fix][UI Next][V1.0.0-Alpha] fix queue display bug in tenant modal#9064
songjianet merged 2 commits into
apache:devfrom
devosend:fix/tenent-manage-queue

Conversation

@devosend

Copy link
Copy Markdown
Contributor

close #9048

@songjianet songjianet added bug Something isn't working UI ui and front end related labels Mar 21, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #9064 (204afa2) into dev (8bb6971) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #9064      +/-   ##
============================================
+ Coverage     39.75%   39.77%   +0.01%     
- Complexity     4262     4263       +1     
============================================
  Files           808      808              
  Lines         32551    32545       -6     
  Branches       3649     3646       -3     
============================================
+ Hits          12941    12944       +3     
+ Misses        18381    18376       -5     
+ Partials       1229     1225       -4     
Impacted Files Coverage Δ
...dolphinscheduler/remote/future/ResponseFuture.java 81.96% <0.00%> (-1.64%) ⬇️
...e/dolphinscheduler/remote/NettyRemotingClient.java 53.52% <0.00%> (+2.81%) ⬆️
...heduler/api/service/impl/TaskGroupServiceImpl.java 66.97% <0.00%> (+3.49%) ⬆️

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 8bb6971...204afa2. Read the comment docs.

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

@songjianet songjianet left a comment

Copy link
Copy Markdown
Member

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 f5ef410 into apache:dev Mar 22, 2022
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-FE][UI Next][V1.0.0-Alpha] The queue name always shows the first one in tenant edit

3 participants