Skip to content

Comments

Fix swagger for Range and Domain#1748

Merged
akshaymankar merged 4 commits intodevelopfrom
akshaymankar/swagger-range-and-domain
Sep 9, 2021
Merged

Fix swagger for Range and Domain#1748
akshaymankar merged 4 commits intodevelopfrom
akshaymankar/swagger-range-and-domain

Conversation

@akshaymankar
Copy link
Member

@akshaymankar akshaymankar commented Sep 9, 2021 •
edited
Loading

Checklist

  • The PR Title explains the impact of the change.
  • The PR description provides context as to why the change should occur and what the code contributes to that effect. This could also be a link to a JIRA ticket or a Github issue, if there is one.
  • If end-points have been added or changed: the endpoint / config-flag checklist (see Wire-employee only backend wiki page) has been followed.
  • If a schema migration has been added, I ran make git-add-cassandra-schema to update the cassandra schema documentation.
  • Section Unreleased of CHANGELOG-draft.md contains the following bits of information:
    • A line with the title and number of the PR in one or more suitable sub-sections.
    • If /a: measures to be taken by instance operators.
    • If /a: list of cassandra migrations.
    • If public end-points have been changed or added: does nginz need upgrade?
    • If internal end-points have been added or changed: which services have to be deployed in a specific order?

@akshaymankar akshaymankar marked this pull request as ready for review September 9, 2021 09:09
pcapriotti
pcapriotti approved these changes Sep 9, 2021
View reviewed changes
Copy link
Contributor

@pcapriotti pcapriotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@akshaymankar @pcapriotti
Remove unnecessary constraints
4a696dd 
Co-authored-by: Paolo Capriotti <paolo@capriotti.io>
@akshaymankar akshaymankar merged commit cadb303 into develop Sep 9, 2021
@akshaymankar akshaymankar deleted the akshaymankar/swagger-range-and-domain branch September 9, 2021 13:46
@jschaul jschaul mentioned this pull request Sep 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pcapriotti pcapriotti pcapriotti approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@akshaymankar @pcapriotti