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.

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

[REST-compliance] Use request body options to merge similar APIs #1274

Closed
hdurix opened this issue Apr 8, 2022 · 2 comments
Closed

[REST-compliance] Use request body options to merge similar APIs #1274

hdurix opened this issue Apr 8, 2022 · 2 comments
Labels
area: enhancement 🔧 generator: internal theme: api

Comments

@hdurix
Copy link
Member

hdurix commented Apr 8, 2022

Following #144

maybe try try to thing about using request bodies to specify some options like:

  • /api/servers/spring-boot/banner with a required request body attribute of the wanted type instead of /api/servers/spring-boot/banner/jhipster-v2 & others
  • merge /api/servers/sonar/java-backend and /api/servers/sonar/java-backend-and-frontend in /api/servers/sonar (btw, sonar is not a server, this could also be changed)

This is to have a self explaining API. It's easier to list the different options allowed.
@hdurix hdurix changed the title [REST-compliance] [REST-compliance] Use request body options to merge similar APIs Apr 8, 2022
@hdurix hdurix mentioned this issue Apr 8, 2022
Closed
@DamnClin
Copy link
Collaborator

Is it still true with the modules APIs?

@pascalgrimaud
Copy link
Member

let's close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area: enhancement 🔧 generator: internal theme: api
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pascalgrimaud @hdurix @DamnClin