Skip to content
This repository was archived by the owner on May 14, 2025. It is now read-only.

Change RequestParam to QueryParam for in api-guide.adoc #6008

Closed
cppwfs opened this issue Oct 23, 2024 · 0 comments · Fixed by #6012 or #6024
Closed

Change RequestParam to QueryParam for in api-guide.adoc #6008

cppwfs opened this issue Oct 23, 2024 · 0 comments · Fixed by #6012 or #6024
Assignees
Milestone

Comments

@cppwfs
Copy link
Contributor

cppwfs commented Oct 23, 2024

With the boot 3.x the tests that were changed to use queryParameters instead of requestParameters will produce query-parameters.adoc vs request-parameters.adoc. This causes Unresolved directive in api-guide.adoc errors in the generated docs.

For all tests in docs that switched to use queryParameters will need their associated entry in api-guide.adoc updated to query-parameters.adoc and changed the associated header as well from Request Parameters to Query Parameters

@cppwfs cppwfs self-assigned this Oct 23, 2024
@cppwfs cppwfs added this to the 3.0.x milestone Oct 23, 2024
corneil pushed a commit to corneil/spring-cloud-dataflow that referenced this issue Oct 24, 2024
Update request-parameters to query-parameters
Added optional where applicable
Added snippet template to query-parameters and updated template for path-parameters to improve readability.

Fixes spring-attic#6008
corneil pushed a commit to corneil/spring-cloud-dataflow that referenced this issue Oct 25, 2024
Added query-parameters and path-parameters where applicable.

Added snippet template to query-parameters and updated template for path-parameters to improve readability.

Fixes spring-attic#6008
@cppwfs cppwfs assigned corneil and unassigned cppwfs Oct 25, 2024
corneil pushed a commit that referenced this issue Oct 28, 2024
* Fix rest documentation for changes to spring-rest-docs

Update request-parameters to query-parameters
Added optional where applicable
Added snippet template to query-parameters and updated template for path-parameters to improve readability.

Fixes #6008

* Update from review

Fixed missing optional
Removed print()
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.