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

Added column to tag headers as required on swagger #161

Merged
merged 4 commits into from
Mar 9, 2023
Merged

Added column to tag headers as required on swagger #161

merged 4 commits into from
Mar 9, 2023

Conversation

RubenBG7
Copy link
Contributor

No description provided.

@RubenBG7 RubenBG7 mentioned this pull request Feb 27, 2023
Closed
@eric-murray
Copy link
Contributor

I made a comment in Issue #124 relevant to this PR.

RubenBG7 and others added 2 commits March 7, 2023 16:22
@RubenBG7
Updated Architecture headers tittle
19fd437 
Deleted standard headers from list due to misunderstanding possibility with the rest of standard http headers not included. Renamed description and clarification of situation in the functionality of the rest of the list
@eric-murray
Update API-design-guidelines.md
423ea10 
- Add `Host` header to section 3.5
- Add statement that headers in section 3.5 should not be included in OAS definition
- Clarify usage of architecture headers
@eric-murray
Copy link
Contributor

I update the PR to include the other points noted in Issue #124

@eric-murray eric-murray merged commit 4be4597 into main Mar 9, 2023
@eric-murray eric-murray deleted the RubenBG7-patch-8 branch April 20, 2023 12:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@eric-murray eric-murray eric-murray approved these changes

@shilpa-padgaonkar shilpa-padgaonkar shilpa-padgaonkar approved these changes

@jordonezlucena jordonezlucena Awaiting requested review from jordonezlucena

Assignees
No one assigned
Labels
commonalities
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RubenBG7 @eric-murray @shilpa-padgaonkar