-
Notifications
You must be signed in to change notification settings - Fork 66
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
Remove the project type when submitting updates to pyxis #768
Remove the project type when submitting updates to pyxis #768
Conversation
Name string `json:"name"` // required | ||
ProjectStatus string `json:"project_status"` // required | ||
Type string `json:"type" default:"Containers"` // required | ||
Name string `json:"name"` // required | ||
ProjectStatus string `json:"project_status"` // required | ||
Type string `json:"type,omitempty"` // required |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A note to reviewers: These lines all changed because the comment's alignment changed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Signed-off-by: Jose R. Gonzalez <jose@flutes.dev>
a4926d2
to
9b62c34
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: acornett21, komish, skattoju The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Fixes #766
The issue listed above results from our CertProject struct, which contains the project type. Certain values for the project's type cannot be set via the external API, but our submission included them, resulting in a rejection from the API.
This patch adjusts the CertProject.Type key such that it uses omitempty, and on update calls, we zero out the value of CertProject.Type before we submit.
We'll want to cut a beta release after this merges, which can then be tested by the blocked partner.
Signed-off-by: Jose R. Gonzalez jose@flutes.dev