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.

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

Implement support for OGC API Features #25

Merged
merged 15 commits into from
Nov 15, 2023
Merged

Conversation

SpeckiJ
Copy link
Contributor

@SpeckiJ SpeckiJ commented Nov 13, 2023

@SpeckiJ SpeckiJ added the enhancement New feature or request label Nov 13, 2023
@SpeckiJ SpeckiJ requested a review from SebaDro November 13, 2023 12:34
@SpeckiJ SpeckiJ self-assigned this Nov 13, 2023
@SebaDro SebaDro changed the base branch from master to develop November 13, 2023 13:42
Copy link
Collaborator

Choose a reason for hiding this comment

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

Better remove this POJO. It may cause interferences with org.springframework.http.ResponseEntity.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there a way to exclude this file from auto-generation? I don't like the workflow of manually deleting the file everytime we regenerate the api-specs.

If this is not trivially possible i would just leave the file there. During development you will very quickly notice that you imported the wrong ResponseEntity as it does not provide the same methods and is a totally different type of class.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes. Just add the class name to the .openapi-generator-ignore file: https://github.com/OpenAPITools/openapi-generator/blob/master/docs/customization.md#ignore-file-format

Copy link
Collaborator

Choose a reason for hiding this comment

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

Where is the test^^?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Besides that, I noticed that the Unit Test fails which leads to a failing build.

Copy link
Collaborator

@SebaDro SebaDro left a comment

Choose a reason for hiding this comment

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

There seems to be a problem qith formatting the bbox query parameter. Can you have a look on it?

@SebaDro SebaDro self-requested a review November 15, 2023 10:15
@SebaDro SebaDro merged commit b11a204 into develop Nov 15, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants