Issue #3622
closed
swagger docs include semantic error
Status:
CLOSED - CURRENTRELEASE
Description
When trying to generate an api bindings at http://editor.swagger.io/#/ i noticed:
Semantic error at definitions.Artifact.required.0
Read only properties cannot be marked as required by a schema.
Jump to line 1504
definitions:
Artifact:
required:
- file <--------------- THIS LINE
type: object
properties:
- Triaged changed from No to Yes
- Sprint set to Sprint 36
- Triaged changed from Yes to No
- Sprint deleted (
Sprint 36)
Do we want to make file not read only or do we want to make it not required?
- Triaged changed from No to Yes
- Sprint set to Sprint 36
- Sprint changed from Sprint 36 to Sprint 37
- Status changed from NEW to ASSIGNED
- Assignee set to dalley
@Austin I think it needs to be not read-only. I just don't understand why it is read-only, I don't see anything setting it to be such.
- Status changed from ASSIGNED to NEW
- Assignee deleted (
dalley)
- Status changed from NEW to ASSIGNED
- Assignee set to bizhang
- Status changed from ASSIGNED to POST
- Sprint changed from Sprint 37 to Sprint 38
- Status changed from POST to MODIFIED
- Sprint/Milestone set to 3.0.0
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Also available in: Atom
PDF
Set parsers explicitly for Artifact Viewset
Uploading files cannot be done using JSON, this disable support for the application/json media_type, Because a file upload is required to create an artifact.
This does not need to be done for Remotes, because files on a remote is optional.
closes #3622 https://pulp.plan.io/issues/3622