camunda / camunda-bpm-platform

Flexible framework for workflow and decision automation with BPMN and DMN. Integration with Quarkus, Spring, Spring Boot, CDI.
https://camunda.com/
Apache License 2.0
4.11k stars 1.55k forks source link

Missing pagination parameters in OpenAPI specification #3715

Closed danielkelemen closed 1 year ago

danielkelemen commented 1 year ago

Environment (Required on creation)

All supported versions. Support case mentioned 7.18 and 7.19.

Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket)

Pagination parameters (firstResult, maxResults) are missing for /history/incident in our OpenAPI specification. REST API supports them.

Steps to reproduce (Required on creation)

Check out the OpenAPI spec (or download it) for /history/incident.

Observed Behavior (Required on creation)

Pagination parameters are missing.

Expected behavior (Required on creation)

Pagination parameters are included.

Root Cause (Required on prioritization)

Pagination params need to be added in the specification.

Solution Ideas

Add <#include "/lib/commons/pagination-params.ftl" >.

Hints

Check the other APIs, if they also lack these parameters.

Links

Breakdown

### Pull Requests
- [ ] https://github.com/camunda/camunda-bpm-platform/pull/3825
- [ ] https://github.com/camunda/camunda-bpm-platform-maintenance/pull/1086
- [ ] https://github.com/camunda/camunda-bpm-platform-maintenance/pull/1087
- [ ] https://github.com/camunda/camunda-bpm-platform-maintenance/pull/1088
- [ ] https://github.com/camunda/camunda-docs-manual/pull/1551

Dev2QA handover

yanavasileva commented 1 year ago

Returning for implementing review hints.

@mboskamp 🔧 On version 7.18 the Rest docs are still in docs-manual, therefore the pagination parameters won't be picked up automatically from the OpenAPI and they are missing there as well:

mboskamp commented 1 year ago

I realized that some POST count endpoints have non-supported and nonsensical pagination and sorting parameters introduced by accident because we use inheritance for the DTO templates. I will try to introduce parameters to the templates that control whether or not to include pagination and/or sorting options.

Edit: This is out of scope for now. I created https://github.com/camunda/camunda-bpm-platform/issues/3878 to track this issue separately.

yanavasileva commented 1 year ago

Review done, thank you for following the review hints!