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(spotify): fixing invalid spec #130

Merged
merged 1 commit into from
Jul 25, 2023
Merged

fix(spotify): fixing invalid spec #130

merged 1 commit into from
Jul 25, 2023

Conversation

ethanwinters
Copy link
Member

@ethanwinters ethanwinters commented Jun 8, 2023

Stricter validation of OpenAPI specs uncovered some errors in this spec around using object patterns for arrays (using "properties" instead of "items"). Also run linting to fix formatting issues.

Stricter validation of OpenAPI specs uncovered some errors in this spec.
"type": "object",
"properties": {
"track": {
"type": "object",
Copy link
Member Author

Choose a reason for hiding this comment

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

This schema gets pretty big and crazy, so leaving it as "object".

Copy link
Contributor

@anirudhaBapat anirudhaBapat left a comment

Choose a reason for hiding this comment

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

Tested this by uploading it in the tooling & it got processed correctly, I haven't tested end to end though because of the lack of an API key. This change looks good to me though as I verified reqest / response params on the UI and actual open api json file.

Copy link

@arrraichu arrraichu left a comment

Choose a reason for hiding this comment

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

Looks good to me! Will merge this by noon to allow for other reviewers to look at 🙂

@ethanwinters ethanwinters merged commit f9b5f5d into master Jul 25, 2023
1 check passed
@ethanwinters ethanwinters deleted the fix-spotify-spec branch July 25, 2023 14:18
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.

None yet

4 participants