Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(spec/search): add vault endpoints to spec (APIC-196) #35

Merged
merged 16 commits into from
Dec 17, 2021

Conversation

DevinCodes
Copy link
Contributor

This adds the spec for all vault endpoints.

@DevinCodes DevinCodes marked this pull request as ready for review December 13, 2021 16:42
@shortcuts shortcuts requested review from a team and damcou and removed request for a team December 13, 2021 16:58
Copy link
Contributor

@damcou damcou left a comment

Choose a reason for hiding this comment

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

Thanks @DevinCodes , looks good, just need a few changes.
Could you also generate the client with yarn generate and commit it please ?
It could also be perfect to add the related tests in the CTS directory please 🙏

Copy link
Contributor

@damcou damcou left a comment

Choose a reason for hiding this comment

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

Sorry I missed some stuff last time :( .
Could you run yarn generate then add some tests on the CTS directory please ? :)

@shortcuts shortcuts requested a review from damcou December 17, 2021 10:48
shortcuts
shortcuts previously approved these changes Dec 17, 2021
damcou
damcou previously approved these changes Dec 17, 2021
Copy link
Contributor

@damcou damcou left a comment

Choose a reason for hiding this comment

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

Looks perfect overall, except a small typo.

@shortcuts shortcuts dismissed stale reviews from damcou and themself via a7991bb December 17, 2021 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants