Skip to content

Conversation

S66D
Copy link

@S66D S66D commented Sep 5, 2022

Defining grouping in swagger endpoints to reduce the challenges brought in creating a per class request models in huge system for better maintainability and for less class redundancy.

@S66D S66D changed the title Defining endpoints grouping in swagger Defining grouping in swagger Sep 5, 2022
@didjoman
Copy link

That Pull-Request would be SO helpful as it fixes a major bug (and thus lot of issues).
I was so happy when I saw it, and I am watching it for 4months now, but sadly it never gets approved :(
How could we fasten the approval of the PR :) ?

@mdaumen
Copy link

mdaumen commented Jan 26, 2023

I'm waiting on this too - it's maddening. I have a model class like this:

  @Schema(description = "Unique ksuid for this Event - returned on submission.", required = false)
  @NotEmpty(groups = {EventConsumerValidations.class})
  @Builder.Default
  private Optional<String> id = Optional.empty();

The idea is we want the validation after the object has been created with an id and not to have a client pass one in. I've tried required = false, I've tried a distinct @Schema annotation on the controller method that specifically omits that id field from the requiredProperties attribute. It's a huge omission. What is the hold-up with this PR?

@virgilsb
Copy link

I'm also looking forward to this.

@b-smets
Copy link

b-smets commented May 8, 2023

Any updates on this PR? Is there something I can assist with to get this merged?

@kangaroo1122
Copy link

Any updates?

@CatarinaRibas
Copy link

Do you have any updates on this?
Can I help with something in order to merge this? We really need this functionality.

Thank you in advance

@Mitic96
Copy link

Mitic96 commented Feb 1, 2024

Any updates?

This is a crucial functionality that would help many of us.
If there is anything that needs to be done before merging this PR, I propose for Maintainers to write the additional requirements.

@lisi9988
Copy link

lisi9988 commented Jul 1, 2024

I also find this very useful as it reduces unnecessary explanations. This is an example I created based on version 1.7.0 and using another project that beautifies the page. I hope the native project will support this feature soon.
knife4j-group, knife4j

@BBeeker-BAnz
Copy link

BBeeker-BAnz commented Aug 5, 2024

Any updates for this?
Here is a solution for a problem sitting for 2 years, why is this still an issue?

@mathiewz
Copy link

Any updates ?
How does this PR still require review since 2022 ? There are discussions about this exact topic since at least 2020... 😐

@Milena92
Copy link

Any updates ? How does this PR still require review since 2022 ? There are discussions about this exact topic since at least 2020... 😐

Indeed.... It's really necessary feature.... Workarounds are painful really for this...

@Gianpiero-Errigo
Copy link

Could we just know if this merge is being considered or is going to be closed, please?

@frantuma
Copy link
Contributor

frantuma commented May 8, 2025

Groups support has been added in #4853

@frantuma frantuma closed this May 8, 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.