Skip to content

[PROD-13928] Make Runner.datasetList optional again #807

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.

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

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

sjoubert
Copy link
Contributor

Requiring it breaks the updateRunner endpoint, or at minima forces users to provide that field (with current values) even if that's not the field they want to modify.
This should be fixed in a better way when we find the time to rework the API contract and split the data model for creation, update and reading.

Copy link
Member

@jreynard-code jreynard-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Requiring it breaks the updateRunner endpoint, or at minima forces users
to provide that field (with current values) even if that's not the field
they want to modify.
This should be fixed in a better way when we find the time to rework the
API contract and split the data model for creation, update and reading.
@sjoubert sjoubert force-pushed the revert_runner_datasetlist_requirement_PROD-13928 branch from 8367b07 to 4ba1abd Compare September 25, 2024 13:09
@sjoubert sjoubert merged commit 4ba1abd into main Sep 25, 2024
33 checks passed
@sjoubert sjoubert deleted the revert_runner_datasetlist_requirement_PROD-13928 branch September 25, 2024 13:48
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.

2 participants