Skip to content
This repository was archived by the owner on Nov 29, 2023. It is now read-only.

Paul schemas sippeers move tns & termination #699

Merged
merged 6 commits into from
Sep 6, 2022

Conversation

pisaychuk
Copy link
Contributor

⚠️Please do not make any changes to spec files in this repository!⚠️

All spec changes should be done in the api-specs repository.

Please follow the Guide for contributing to our developer documentation.

@pisaychuk pisaychuk requested a review from a team September 2, 2022 11:47
}
}
},
"/sippeers/products/termination/accounts/{accountId}/sites/{siteId}/sippeers/{sipPeerId}/products/termination": {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this the right path? Shouldn't it be /accounts/{accountId}/sites/{siteId}/sippeers/{sipPeerId}/products/termination

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, thanks @mrmeyers99, I missed the url-rewrite logic here, addressing

brianluisgomez
brianluisgomez previously approved these changes Sep 6, 2022
@pisaychuk
Copy link
Contributor Author

@mrmeyers99 comment is addressed, paths have been fixed. @brianluisgomez would ask for re-approve, thanks!

@mrmeyers99 mrmeyers99 merged commit 1a6ee33 into Bandwidth:main Sep 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants