camaraproject / Commonalities

Repository to describe, develop, document and test the common guidelines and assets for CAMARA APIs
Apache License 2.0
9 stars 24 forks source link

API Design Guidelines chapter 11: 'info' object, 'servers' object and cleanup #214

Closed rartych closed 3 weeks ago

rartych commented 1 month ago

What type of PR is this?

Add one of the following kinds:

What this PR does / why we need it:

The definitions of 'info' and 'servers` object were added to chapter 11. Other edits on API definition guidelines. I propose to remove the figures as they do not add value to the text.

Which issue(s) this PR fixes:

Fixes #199 Fixes #201

Special notes for reviewers:

Info object described in the same manner as here: https://swagger.io/docs/specification/api-general-info/

Changelog input

The definitions of 'info' and 'servers` object were added to chapter 11

Additional documentation

shilpa-padgaonkar commented 1 month ago

@patrice-conil , @PedroDiez and @RubenBG7 : Could you kindly review & approve this if all looks ok? we are trying to speed up reviews of at least the straightforward/simple PRs so that we are in time for the 0.4 release. Thanks in advance.

shilpa-padgaonkar commented 1 month ago

@PedroDiez or @RubenBG7 Could one of you kindly approve this PR (if you find no issues) so that we can proceed to merge? Thanks in advance.

tanjadegroot commented 1 month ago

Hi Rafal,

Yes that is indeed the assumption. It is a good idea to have that I the example and guidelines.

Cheers Tanja

From: Rafal Artych @.> Sent: Tuesday, June 4, 2024 9:25 AM To: camaraproject/Commonalities @.> Cc: Tanja De Groot (Nokia) @.>; Mention @.> Subject: Re: [camaraproject/Commonalities] API Design Guidelines chapter 11: 'info' object, 'servers' object and cleanup (PR #214)

@rartych commented on this pull request.


In documentation/API-design-guidelines.mdhttps://github.com/camaraproject/Commonalities/pull/214#discussion_r1625488344:

  • title without "API" in it, e.g. "Number Verification"

I think it is assumed that the contact information will be changed by each API Provider (Operator or Aggregator) when publishing API specification as the documentation for their service - possibly it should be explained in API Design Guidelines here in chapter 11. When yaml is hosted in CAMARA Github all contact info is in main README.md @tanjadegroothttps://github.com/tanjadegroot @hdamkerhttps://github.com/hdamker WDYT?

— Reply to this email directly, view it on GitHubhttps://github.com/camaraproject/Commonalities/pull/214#discussion_r1625488344, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AU6LGAZ6ZBFV7UTVBSLRGYLZFVTWHAVCNFSM6AAAAABIHRMHLGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOJVGU2DAOBQGU. You are receiving this because you were mentioned.Message ID: @.**@.>>

rartych commented 1 month ago

Following the discussion in Release Management: https://wiki.camaraproject.org/display/CAM/2024-06-04+Release+WG+Minutes I have added x-camara-commonalities extension field to the info object.

patrice-conil commented 1 month ago

LGTM