Skip to content

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

3 participants