komsa-ag / camunda.api.client Goto Github PK

https://docs.camunda.org/manual/latest/reference/rest/external-task/post-lock/

POST /external-task/{id}/lock

Hello, in my project I need to update and delete users. I'm ready with MR to add this methods.

according to https://docs.camunda.org/manual/7.13/reference/rest/history/variable-instance/get-variable-instance/ there should be a CreateTime Property, unfortunately the client doesn't expose it.

I can fix this easily, but I'd like confirmation from the existing team before I do so...

This methodology requires public interfaces:
https://github.com/reactiveui/refit#reducing-header-boilerplate-with-delegatinghandlers-authorization-headers-worked-example

But the client has all internal interfaces:
Example: internal interface IExternalTaskRestService

Making these external would allow more flexible uses of the nuget package.

new API-Features:

https://docs.camunda.org/manual/7.17/reference/rest/deployment/get-registered/

Let's find missing API-Features. Please add comments to this issues, which features are missing

I found an owner in the task, but how do I set it when initiating it? It seems that I have to set the owner every time I go to the next step.

If I search "Camunda.Api.Client", I only get 3 C# repository, this project is not included. But since the original project is not maintained anymore, it's unfair for forked project lives under the shadow of it. Will remove the "forked from" help?

The ExternalTaskLock class should be using the name "LockDuration" instead of "Duration". Currently the External Task lock will fail because it thinks you did not set the lock duration.

Refit 6 makes System.Text.Json the default JSON serializer.
Newtonsoft.Json is slower than System.Text.Json and is replaced in many projects

Missing properties ( lastUpdated and camundaFormRef) they are returned by the api but non existing in the models.

public DateTime? LastUpdated;
public string? CamundaFormRef;

https://docs.camunda.org/manual/7.18/reference/rest/task/get/#result

See title - came over from the original client to find it still missing ... should be

public DateTime? DeployedAfter;

In our case we have literally thousands of workflow-definitions and have to do some checks on a regular basis, right now it takes more than a minute for a call to get all process definition info entries, which we could easily reduce if we just fetch the definitions newer than our last call. Yes, we could probably mitigate that by using the deployments endpoint, but that would mean a lot of different calls for just that information where the api already has a corresponding solution ...