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

fix: open api doc page display #1016

Merged
merged 7 commits into from
Oct 22, 2024
Merged

fix: open api doc page display #1016

merged 7 commits into from
Oct 22, 2024

Conversation

nathanbogale
Copy link
Contributor

The open api docs is now displayed properly and in a full screen

Copy link
Contributor

github-actions bot commented Oct 17, 2024

Test Report | ${\color{lightgreen}Pass: 297/297}$ | ${\color{red}Fail: 0/297}$ |

@nathanbogale nathanbogale self-assigned this Oct 17, 2024
@nathanbogale nathanbogale requested review from nielskijf and removed request for saibatizoku, FelipeRosa, minikin, Mr-Leshiy and cong-or October 17, 2024 08:58
Copy link
Collaborator

@stevenj stevenj left a comment

Choose a reason for hiding this comment

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

Other than the changes to cat-gateway, this PR is fine.
Please revert the cat-gateway changes.

@minikin minikin added the review me PR is ready for review label Oct 18, 2024
stevenj and others added 2 commits October 21, 2024 15:59
The mods file has been taken out as its not needed in in the voices opneapi doc rendering
@nathanbogale nathanbogale requested review from neil-iohk and removed request for nielskijf October 21, 2024 10:51
@nathanbogale nathanbogale requested a review from stevenj October 22, 2024 14:26
@stevenj stevenj merged commit 8ce19a9 into main Oct 22, 2024
33 checks passed
@stevenj stevenj deleted the fix-openapi-doc-display branch October 22, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review me PR is ready for review
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants