Skip to content

Feature | Extend Swagger Coverage for controller ConsultantsApiController #364

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.

Sign up for GitHub

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

Jump to bottom
Merged
smarcet merged 4 commits into from
Oct 30, 2025
Merged

Feature | Extend Swagger Coverage for controller ConsultantsApiController #364

smarcet merged 4 commits into from
Oct 30, 2025
+99 −3

Conversation

@matiasperrone-exo
Copy link
Contributor

@matiasperrone-exo matiasperrone-exo commented Sep 30, 2025 •
edited
Loading

@matiasperrone-exo matiasperrone-exo self-assigned this Sep 30, 2025
@smarcet smarcet force-pushed the main branch 2 times, most recently from 4f4b976 to 9ed27c0 Compare September 30, 2025 17:46
@matiasperrone-exo matiasperrone-exo force-pushed the feature/add-openapi-documentation-to-controller-marketplace---consultantsapicontroller branch from 262757a to 8990807 Compare September 30, 2025 19:14
@matiasperrone-exo matiasperrone-exo marked this pull request as ready for review September 30, 2025 19:34
@smarcet smarcet force-pushed the main branch 9 times, most recently from c94fc68 to 9a8387b Compare October 2, 2025 17:58
@matiasperrone-exo matiasperrone-exo added the documentation Improvements or additions to documentation label Oct 7, 2025
@matiasperrone-exo matiasperrone-exo force-pushed the feature/add-openapi-documentation-to-controller-marketplace---consultantsapicontroller branch from f2c1aa8 to 44ea73b Compare October 7, 2025 21:32
@matiasperrone-exo matiasperrone-exo changed the title Feature | Extend Swagger Coverage for controller Apis/Marketplace/ConsultantsApiController.php Feature | Extend Swagger Coverage for controller ConsultantsApiController Oct 7, 2025
@matiasperrone-exo matiasperrone-exo force-pushed the feature/add-openapi-documentation-to-controller-marketplace---consultantsapicontroller branch from 69bd922 to 45f9678 Compare October 14, 2025 15:59
caseylocker
caseylocker approved these changes Oct 30, 2025
View reviewed changes
Copy link

@caseylocker caseylocker left a comment

Choose a reason for hiding this comment

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

This is approved for merge.

smarcet
smarcet approved these changes Oct 30, 2025
View reviewed changes
Copy link
Collaborator

@smarcet smarcet left a comment

Choose a reason for hiding this comment

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

LGTM

@smarcet smarcet merged commit e912c81 into main Oct 30, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caseylocker caseylocker caseylocker approved these changes

@smarcet smarcet smarcet approved these changes

Assignees

@matiasperrone-exo matiasperrone-exo

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@matiasperrone-exo @caseylocker @smarcet