Skip to content

Fix the bug #1968.#1969

Merged
davidzollo merged 1 commit intoapache:devfrom
nauu:bug#1968
Feb 17, 2020
Merged

Fix the bug #1968.#1969
davidzollo merged 1 commit intoapache:devfrom
nauu:bug#1968

Conversation

@nauu
Copy link
Copy Markdown
Contributor

@nauu nauu commented Feb 17, 2020

What is the purpose of the pull request

Fix the bug #1968.

When create a new task or edit an exist task.The task details window will be open.
Then if delete the task node or click the DataSource menu. And the task details window still open.

For now, the task details window will be close.

Brief change log

  • Edit dag.vue
  • Edit dag.js
  • Edit jsPlumbHandle.js

Verify this pull request

This change added tests and can be verified as follows:

  • Manually verified the change by testing locally.

When create a new task or edit an exist task.The task details window will be open.
Then if delete the task node or click the DataSource menu. And the task details window still open.

The task details window should be close.
@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 17, 2020

Codecov Report

Merging #1969 into dev will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #1969      +/-   ##
============================================
+ Coverage     30.32%   30.32%   +<.01%     
+ Complexity     1370     1368       -2     
============================================
  Files           294      294              
  Lines         14999    14999              
  Branches       1806     1806              
============================================
+ Hits           4548     4549       +1     
+ Misses        10002     9998       -4     
- Partials        449      452       +3
Impacted Files Coverage Δ Complexity Δ
...scheduler/alert/template/AlertTemplateFactory.java 42.85% <0%> (-14.29%) 2% <0%> (ø)
...inscheduler/common/thread/ThreadPoolExecutors.java 80% <0%> (-1.91%) 8% <0%> (-2%)
...er/server/worker/task/AbstractCommandExecutor.java 42.13% <0%> (+2.03%) 14% <0%> (ø) ⬇️

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 9b75beb...b1b8e95. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@break60
Copy link
Copy Markdown
Contributor

break60 commented Feb 17, 2020

+1

@davidzollo davidzollo merged commit afd5c75 into apache:dev Feb 17, 2020
elonlo pushed a commit to fordeal-smalldata/incubator-dolphinscheduler that referenced this pull request Jul 26, 2021
When create a new task or edit an exist task.The task details window will be open.
Then if delete the task node or click the DataSource menu. And the task details window still open.

The task details window should be close.
# Conflicts:
#	dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/dag.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants