Skip to content

Proposal to fix #2531 adding explode modifier * to sort attribute #2945

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

Closed
wants to merge 4 commits into from

Conversation

motofix
Copy link

@motofix motofix commented Sep 28, 2023

Adding explode modifier * to sort attribute, making client libraries like Ketting to behave correctly and generate request compliant with Spring

Proposal to fix #2531

Julien Béti added 2 commits September 28, 2023 11:32
…'sort' parameter template variable

Adding author in header
@pivotal-cla
Copy link

@motofix Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 28, 2023
@pivotal-cla
Copy link

@motofix Thank you for signing the Contributor License Agreement!

mp911de and others added 2 commits October 2, 2023 19:19
…'sort' parameter template variable

Using TemplateVariable.composite
@motofix motofix requested a review from odrotbohm October 2, 2023 17:22
@odrotbohm
Copy link
Member

LGTM, I'll take it from here. Likely by the end of the week. Thanks for contributing this, Julien!

@odrotbohm odrotbohm removed the status: waiting-for-triage An issue we've not yet triaged label Oct 3, 2023
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 3, 2023
@motofix
Copy link
Author

motofix commented Dec 1, 2023

Hi, any news on this ?

@odrotbohm
Copy link
Member

Sorry for the delay and thanks for bumping this. I'll have a loot ASAP, this time for sure, I promise.

odrotbohm pushed a commit that referenced this pull request Dec 1, 2023
HateoasSortHandlerMethodArgumentResolver now renders the sort template variable as composite to properly indicate that it can be submitted multiple times.

Fixes GH-2531.
Original pull request GH-2945.
odrotbohm pushed a commit that referenced this pull request Dec 1, 2023
HateoasSortHandlerMethodArgumentResolver now renders the sort template variable as composite to properly indicate that it can be submitted multiple times.

Fixes GH-2531.
Original pull request GH-2945.
@odrotbohm
Copy link
Member

This is polished and back-ported to the 3.1.x maintenance branch. Thanks again for your patience!

@odrotbohm odrotbohm closed this Dec 1, 2023
@odrotbohm odrotbohm removed the status: waiting-for-triage An issue we've not yet triaged label Dec 1, 2023
odrotbohm pushed a commit that referenced this pull request Dec 1, 2023
HateoasSortHandlerMethodArgumentResolver now renders the sort template variable as composite to properly indicate that it can be submitted multiple times.

Fixes GH-2531.
Original pull request GH-2945.
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.

HateoasSortHandlerMethodArgumentResolver should append * to the 'sort' parameter template variable.
5 participants