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

Click on workflow node navigates to the node definition #3062

Merged
merged 4 commits into from
Aug 23, 2022

Conversation

azriel46d
Copy link

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

This small PR addresses source code navigation. Currently, on larger flows, the author needs to search in the JSON definition to find a particular task. With this PR, clicking a node in the workflow diagram would take advantage of the Monaco Editor to find the taskReferenceName, scroll to it , and highlight it using the editor's Decorations.

Screen.Recording.2022-06-23.at.12.57.15.mov

@jxu-nflx jxu-nflx requested a review from peterlau July 19, 2022 21:13
@peterlau peterlau merged commit 6b8e920 into Netflix:main Aug 23, 2022
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.

3 participants