Updates to Registries API based on tech review

Description

Structural problems:

  • Gitbook

    • there could be more descriptions of parameters/keys to understand their purpose/details

    • two endpoints are duplicated (/data/{registryname}/{versionnumber}/read and /data/{registryname}/{versionnumber})

  • Github 

    • too many files makes the API directory unclear

    • in Readme file it is stated that the standard is json, but it should be yaml

    • json files could be removed

    • /data/{registryname}/{versionnumber}/create is in Swagger but not in Gitbook

Discrepancies and missing info:

  • some parameters follow snake case and some camel case - it should be standardized

  • some endpoints are duplicated

  • one endpoint is only in Swagger

  • almost no error responses are described

API problems:

 

 

 

  • POST /data/{registryname}/{versionnumber}/create

    • it’s only in Swagger, not in Gitbook

    • no error case (example+schema)

Final conclusions:

  • Readme file should be updated

  • Files in API directory should be minimized (all endpoints should be in one file)

  • Parameters/keys in schemas could be better described (purpose/details)

  • Error responses should be added

Activity

Show:

Steve ConradOctober 3, 2023 at 4:06 PM

- please let or myself know if you have questions on the above. Thanks.

Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Priority

Checklist

Created October 3, 2023 at 4:05 PM
Updated October 3, 2023 at 4:06 PM