Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Flow validation should know about taskDefaults #388

Closed
tchiotludo opened this issue Oct 19, 2021 · 0 comments · Fixed by #871
Closed

Flow validation should know about taskDefaults #388

tchiotludo opened this issue Oct 19, 2021 · 0 comments · Fixed by #871
Assignees
@Skraye
Labels
area/backend Needs backend code changes bug Something isn't working

Comments

@tchiotludo
Copy link
Member

If a flow have a taskDefaults and there is a mandatory properties on a task, right now micronaut validation will lead this on due to @Body annotation.
Maybe a better approach will be to inject taskDefaults and after trigger the validation.
It will also help for this one :
@tchiotludo tchiotludo added bug Something isn't working area/backend Needs backend code changes labels Oct 19, 2021
@tchiotludo tchiotludo moved this to Backlog in All issues Oct 19, 2021
@tchiotludo tchiotludo moved this from Backlog to Ready in All issues Oct 29, 2021
@tchiotludo tchiotludo mentioned this issue Apr 29, 2022
Closed
@Skraye Skraye mentioned this issue Jan 16, 2023
Merged
@tchiotludo tchiotludo moved this from Ready to In Progress in All issues Jan 24, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in All issues Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Skraye Skraye

Labels
area/backend Needs backend code changes bug Something isn't working
Projects
No open projects
All issues
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Flows validation kestra-io/kestra
2 participants
@tchiotludo @Skraye