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

docs: boolean filter #2124

Closed
wants to merge 46 commits into from
Closed

Conversation

adryyy10
Copy link

Adding boolean filter docs

noweh and others added 19 commits October 12, 2024 19:03
- Added missing 'use' statements in the laravel/index.md for clarity
- Fixed a typo in a Model usage
- Corrected 'ProviderInterface::class' to 'provider' in ApiServiceProvider
Change 'provider' to ProviderInterface::class in laravel/index.md
Fix some docs about DTO
Use the constructor property promotion syntax in ApiResource/Book
* Update laravel/index.md file

- Added missing 'use' statements in the laravel/index.md for clarity
- Fixed a typo in a Model usage
- Corrected 'ProviderInterface::class' to 'provider' in ApiServiceProvider

* Update index.md

Change 'provider' to ProviderInterface::class in laravel/index.md

* Update laravel/index.md

Fix some docs about DTO

* Update laravel/index.md

Use the constructor property promotion syntax in ApiResource/Book

---------

Co-authored-by: Antoine Bluchet <[email protected]>
Updated the code block from 'http request' to 'http' for proper syntax highlighting.
* Update bootstrap.md

Fix typo

* Update core/bootstrap.md

---------

Co-authored-by: Vincent Amstoutz <[email protected]>
Update link to Typescript interfaces. The old one is pointing to a deprecated handbook page.
* fix: Symfony service controller reference for "api_doc" route

* fix: laravel service controller reference for "api_doc" route

---------

Co-authored-by: Vincent Amstoutz <[email protected]>
* Update index.md

Fix ambiguous choices

* fix url
@adryyy10
Copy link
Author

@vinceAmstoutz Added docs for BooleanFilter

@vinceAmstoutz
Copy link
Contributor

@adryyy10 Thanks for your promptness! But as I asked, can you please target main?

@adryyy10 adryyy10 changed the base branch from 4.0 to main January 10, 2025 10:37
@vinceAmstoutz
Copy link
Contributor

vinceAmstoutz commented Jan 10, 2025

Note that rebasing is required @adryyy10

noweh and others added 7 commits January 10, 2025 11:22
- Added missing 'use' statements in the laravel/index.md for clarity
- Fixed a typo in a Model usage
- Corrected 'ProviderInterface::class' to 'provider' in ApiServiceProvider
Change 'provider' to ProviderInterface::class in laravel/index.md
Fix some docs about DTO
Use the constructor property promotion syntax in ApiResource/Book
vinceAmstoutz and others added 20 commits January 10, 2025 11:22
Updated the code block from 'http request' to 'http' for proper syntax highlighting.
* Update bootstrap.md

Fix typo

* Update core/bootstrap.md

---------

Co-authored-by: Vincent Amstoutz <[email protected]>
Update link to Typescript interfaces. The old one is pointing to a deprecated handbook page.
* fix: Symfony service controller reference for "api_doc" route

* fix: laravel service controller reference for "api_doc" route

---------

Co-authored-by: Vincent Amstoutz <[email protected]>
* Update index.md

Fix ambiguous choices

* fix url
- Added missing 'use' statements in the laravel/index.md for clarity
- Fixed a typo in a Model usage
- Corrected 'ProviderInterface::class' to 'provider' in ApiServiceProvider
Change 'provider' to ProviderInterface::class in laravel/index.md
Fix some docs about DTO
Updated the code block from 'http request' to 'http' for proper syntax highlighting.
- Added missing 'use' statements in the laravel/index.md for clarity
- Fixed a typo in a Model usage
- Corrected 'ProviderInterface::class' to 'provider' in ApiServiceProvider
Change 'provider' to ProviderInterface::class in laravel/index.md
Fix some docs about DTO
@vinceAmstoutz
Copy link
Contributor

Or if you prefer, you can close this PR and open another @adryyy10

@adryyy10 adryyy10 closed this by deleting the head repository Jan 10, 2025
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.

10 participants