Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Show tooltip based on task description in UI #1008

Merged
merged 3 commits into from
Apr 2, 2019

Conversation

joshuadeanhall
Copy link
Contributor

When hovering on a task it will show the description of the task to make it easier to understand the purpose of the task in both the workflow definition and execution pages.

@codecov
Copy link

codecov bot commented Feb 23, 2019

Codecov Report

Merging #1008 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev    #1008   +/-   ##
=========================================
  Coverage     62.94%   62.94%           
  Complexity     2615     2615           
=========================================
  Files           231      231           
  Lines         13119    13119           
  Branches       1314     1314           
=========================================
  Hits           8258     8258           
  Misses         4134     4134           
  Partials        727      727

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 f841713...b1f38d4. Read the comment docs.

@coveralls
Copy link

coveralls commented Feb 23, 2019

Pull Request Test Coverage Report for Build 2522

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.007%) to 68.966%

Totals Coverage Status
Change from base Build 2503: 0.007%
Covered Lines: 9258
Relevant Lines: 13424

💛 - Coveralls

@apanicker-nflx
Copy link
Collaborator

@joshuadeanhall Thank you for the contribution. Please rebase the changes against the dev branch and resolve the conflicts so that this can be merged.

When hovering on a task it will show the description of the task to make it easier to understand the purpose of the task in both the workflow definition and execution pages.
@joshuadeanhall
Copy link
Contributor Author

@apanicker-nflx I resolved the conflict and everything appears to be passing.

@kishorebanala kishorebanala merged commit 0dd4f12 into Netflix:dev Apr 2, 2019
long-64 pushed a commit to long-64/conductor that referenced this pull request Oct 2, 2019
…oltip

 Show tooltip based on task description in UI
peterlau pushed a commit that referenced this pull request Sep 1, 2021
 Show tooltip based on task description in UI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants