Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

fix: updated assigned_to to required in ipassignments #160

Closed
wants to merge 1 commit into from
Closed

fix: updated assigned_to to required in ipassignments #160

wants to merge 1 commit into from

Conversation

codinja1188
Copy link
Contributor

No description provided.

@ctreatma
Copy link
Contributor

ctreatma commented Sep 27, 2023 •
edited
Loading

@Vasubabu keep in mind that this cannot work until the SDK is able to return errors when required fields are missing (#132) (#172)

@ctreatma
Copy link
Contributor

Also, TIL that @t0mk took the same approach in metal-python (it works there b/c the Python generator supports required properties): https://github.com/equinix-labs/metal-python/blob/main/scripts/patch_metal_spec.py#L63-L67

@ctreatma ctreatma closed this Jun 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@codinja1188 @ctreatma