Feature/843 provide task list filter criteria #335
Conversation
|
I didn't quite got it. The referenced PR of polyflow was merged but still, I had to fix some compilation errors. On doing so, I realized that the two relevant queries are now missing filters in polyflow... So adopted it in the example code, but then, we should probably adopt in in the Open API too... Please have a look again, @p-wunderlich ... I'll mark the relevant places in the review |
| '/tasks/payload/attribute/names': | ||
| parameters: | ||
| - $ref: '#/components/parameters/CurrentUserIdParam' | ||
| - $ref: '#/components/parameters/FiltersParam' |
There was a problem hiding this comment.
There are no filters in the api anymore
| parameters: | ||
| - $ref: '#/components/parameters/CurrentUserIdParam' | ||
| - $ref: '#/components/parameters/AttributeNameParam' | ||
| - $ref: '#/components/parameters/FiltersParam' |
| override fun getTasksAttributeValues( | ||
| xCurrentUserID: String, | ||
| attributeName: String, | ||
| filters: List<String>? // FIXME -> no filters needed? |
There was a problem hiding this comment.
As you see, the generated method signature is not matching the API call in line 169
| .query( | ||
| TaskAttributeNamesQuery( | ||
| user = user, | ||
| assignedToMeOnly = false, |
There was a problem hiding this comment.
because there are no filters in the query object
| TaskAttributeValuesQuery( | ||
| attributeName = attributeName, | ||
| user = user, | ||
| assignedToMeOnly = false, |
There was a problem hiding this comment.
because there are no filters in the query object
Should be done AFTER #323 and new Polyflow release is needed