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

Review API readiness minimum criteria doc #5

Closed
shilpa-padgaonkar opened this issue Jan 26, 2022 · 9 comments
Closed

Review API readiness minimum criteria doc #5

shilpa-padgaonkar opened this issue Jan 26, 2022 · 9 comments

Comments

@shilpa-padgaonkar
Copy link
Contributor

shilpa-padgaonkar commented Jan 26, 2022

https://github.com/camaraproject/WorkingGroups/blob/main/Commonalities/documentation/Working/API-Readiness-Checklist.md

@jordonezlucena
Copy link
Contributor

API readiness minimum criteria should define the minimum set of requirements that allow going from step 3 (implementation) to step 4 (validation). If my interpretation is correct, here are my comments:

  1. API spec: whether OAS3 or Swagger2 depends on the outcome of issue camaraproject/rep_main#29
  2. API implementation: it is tagged as optional (not mandatory). How is it possible to validate an API that we do not have a reference implementation of? What is the value of having a service API if there is no source code?
  3. Agree
  4. Agree
  5. Agree -> Do we need to agree on a specific API Test Plan (ATP) format? Does this need to be captured in the generic principles doc?
  6. Agree

@shilpa-padgaonkar
Copy link
Contributor Author

  1. OAS3/Swagger2 is part of https://github.com/camaraproject/rep_main/issues/29 . API doc template only mentions that specification section needs to be part of API documentation.
  2. This reflects the decision announced in the larger Camara alliance round. In WG, we can elaborate further if needed.
  3. This we hope can be addressed in https://github.com/camaraproject/rep_main/issues/28

@petorre
Copy link

petorre commented Feb 10, 2022

Relationship between Implementation and Test Cases: Test Cases will be developed to target some particular Implementation. If this is some jointly agreed Reference Implementation, later that Test Case will give easy answer if some commercial/custom/proprietary implementation A or B is passing of failing to pass test cases. Such answers can be debated if there is no Reference.

@MarkusKuemmerle MarkusKuemmerle transferred this issue from camaraproject/Governance Feb 22, 2022
@T-sm
Copy link
Contributor

T-sm commented Feb 23, 2022

@Kevsy
Copy link
Contributor

Kevsy commented Mar 9, 2022

Created [PR 28] (#28)

  • added security and privacy compliance criteria

@T-sm T-sm added this to the milestone #2 milestone May 2, 2022
@jordonezlucena
Copy link
Contributor

@shilpa-padgaonkar, @T-sm: the "API-Readiness-Checklist.md" needs to be revisited, as the links captured there seem to be broken

@T-sm
Copy link
Contributor

T-sm commented May 4, 2022

@jordonezlucena @shilpa-padgaonkar Thanks, José, the links have been updated. Also the document has been moved to Working directory in line with issue #35 request.

@shilpa-padgaonkar
Copy link
Contributor Author

PR #38 assigned to @Kevsy
If merged, this issue can be closed.

@Kevsy
Copy link
Contributor

Kevsy commented May 18, 2022

PR #38 Merged

@Kevsy Kevsy closed this as completed May 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants