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

Fix validation error handling in workflow/task editor #3779

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

Young-Zen
Copy link
Contributor

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

In workflow/task editor, the UI does not display the error message well when saving fails.
image

After fixing the bug, the UI can display detailed error message:
image

image

Alternatives considered

Describe alternative implementation you have considered

Signed-off-by: yanghaojia <2453883990@qq.com>
@Young-Zen Young-Zen force-pushed the ui-fix-validation-error-handling branch from 9efd256 to a68016e Compare September 25, 2023 03:58
@Young-Zen
Copy link
Contributor Author

@v1r3n Hello, the CI build failure was not caused by my changes in the PR, it might have been caused by other. What should I do to push my PR through?

@v1r3n v1r3n merged commit 22ad050 into Netflix:main Sep 27, 2023
2 checks passed
Young-Zen pushed a commit to Young-Zen/netflix-conductor that referenced this pull request Nov 10, 2023
…r-handling

Fix validation error handling in workflow/task editor
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.

2 participants