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.

Sign up for GitHub

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

Jump to bottom

Add missing "Byron Assets" heading to OpenAPI docs #2524

Merged
merged 1 commit into from
Feb 16, 2021
Merged

Add missing "Byron Assets" heading to OpenAPI docs #2524

merged 1 commit into from
Feb 16, 2021

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Feb 16, 2021 •
edited
Loading

Issue Number

ADP-603

Overview

All the /byron-wallets/{walletId}/assets endpoints were missing from the OpenAPI docs. This fixes it.

Comments

Rendered version

@rvl rvl self-assigned this Feb 16, 2021
KtorZ
KtorZ approved these changes Feb 16, 2021
View reviewed changes
Copy link
Member

@KtorZ KtorZ left a comment

Choose a reason for hiding this comment

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

Whoops.

@piotr-iohk
Copy link
Contributor

whoops.

@nikolaglumac
Copy link

200_d

@KtorZ
Copy link
Member

KtorZ commented Feb 16, 2021

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Feb 16, 2021

Build succeeded:

@iohk-bors iohk-bors bot merged commit 8bde7ed into master Feb 16, 2021
@iohk-bors iohk-bors bot deleted the rvl/adp-603/fix-swagger-byron-assets branch February 16, 2021 14:37
@rvl
Copy link
Contributor Author

rvl commented Feb 16, 2021

The documentation was always there, if you can read swagger.yaml files 😝

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KtorZ KtorZ KtorZ approved these changes

Assignees

@rvl rvl

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rvl @piotr-iohk @nikolaglumac @KtorZ