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(notification): Include notification details #318

Merged
merged 3 commits into from
Apr 24, 2024

Conversation

ttshivers
Copy link
Contributor

Ensure the notification details schemas are included in the bundled spec.

Update to @redocly/cli which replaces @redocly/openapi-cli.
https://www.npmjs.com/package/@redocly/openapi-cli

This is necessary to bring in fixes for the bundler not properly including refs when adding extra schemas.

Fixes: #317

Update to @redocly/cli which replaces @redocly/openapi-cli.
https://www.npmjs.com/package/@redocly/openapi-cli

This is necessary to bring in fixes for the bundler not properly
including refs when adding extra schemas.
Include the notification details schemas manually since they aren't
referenced directly anywhere else.

Fixes: vrchatapi#317
@ttshivers ttshivers force-pushed the include-notification-details branch from 5efff96 to ef4f957 Compare April 24, 2024 22:14
Add ignore for unused component
@ariesclark ariesclark merged commit 1b8686b into vrchatapi:main Apr 24, 2024
6 checks passed
@ttshivers ttshivers deleted the include-notification-details branch April 24, 2024 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Notification details types aren't included
2 participants