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 filter and collection docs links #1868

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

mikerebilly
Copy link
Contributor

@mikerebilly mikerebilly commented Apr 11, 2024

Summary

Remove old documentation about filters and expand, and link to new docs. Also fixed sort definition and description.

Links

Checklist

  • Writing style
  • API design standards

@mikerebilly mikerebilly self-assigned this Apr 11, 2024
@mikerebilly mikerebilly marked this pull request as ready for review April 11, 2024 14:21
@mikerebilly mikerebilly requested review from a team as code owners April 11, 2024 14:22
@mikerebilly mikerebilly merged commit ad7d35a into main Apr 11, 2024
2 checks passed
@mikerebilly mikerebilly deleted the link-filter-collections-dev-docs branch April 11, 2024 14:34
Comment on lines 9 to -11
schema:
type: array
items:
type: string
Copy link
Contributor

Choose a reason for hiding this comment

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

@mikerebilly could you explain me this change? I believe it's incorrect.
Please see specification for style: form + explode: false

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@justnero You're right seems like i'm wrong, maybe we can revert this specific part of the PR. https://swagger.io/specification/#style-values:~:text=200.B%3D150-,form,false,-color%3D cc @mehicalen

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mikerebilly mikerebilly mentioned this pull request Apr 11, 2024
2 tasks
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.

5 participants