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

Update to OpenAPI spec v3.1.0 #910

Closed
wants to merge 1 commit into from

Conversation

hopsoft
Copy link

@hopsoft hopsoft commented Sep 12, 2024

The current implementation uses the deprecated example field which causes test failures whenever creating a new Bullet Train application, and whenever a new Super Scaffold is generated.

This PR updates the implementation to use the current examples field instead.

SEE: https://spec.openapis.org/oas/latest.html#fixed-fields-5

Note

This change satisifies the test failure problem but we should verify that it returns what's truly needed for the OpenAPI docs.

Related: bullet-train-co/bullet_train#1520

The current implementation uses the deprecated field `example` which causes test failures whenever creating a new Bullet Train application, and whenever a new Super Scaffold is generated.

This PR updates the implementation to use the current `examples` field instead.

SEE: https://spec.openapis.org/oas/latest.html#fixed-fields-5
@hopsoft
Copy link
Author

hopsoft commented Sep 12, 2024

Note that I still have a few places to patch up. Working on that now.

@hopsoft hopsoft marked this pull request as draft September 12, 2024 21:21
@hopsoft
Copy link
Author

hopsoft commented Sep 13, 2024

That first stab was an AI recommendation. It points the way... sort of, but I believe the issue is in the bullet-train-co/jbuilder-schema gem. Taking a deeper look.

@hopsoft
Copy link
Author

hopsoft commented Sep 13, 2024

Abandoning this and will leave to the OpenAPI domain experts that did the original implementation.

@hopsoft hopsoft closed this Sep 13, 2024
@hopsoft hopsoft deleted the hopsoft/open-api-3.1.0 branch September 13, 2024 00:24
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.

1 participant