flowable-engine icon indicating copy to clipboard operation
flowable-engine copied to clipboard

Support UUID value type in flowable-rest API (#3856)

Open pvojtechovsky opened this issue 1 year ago • 6 comments

This is fix for #3856

Check List:

  • Unit tests: NO
  • Documentation: NO

pvojtechovsky avatar Mar 04 '24 15:03 pvojtechovsky

Thanks for the PR, can you also add this variable converter to the cmmn and dmn engine REST APIs?

tijsrademakers avatar Apr 11 '24 07:04 tijsrademakers

can you also add this variable converter to the cmmn and dmn engine REST APIs?

Makes sense, good idea. I just did it now.

pvojtechovsky avatar Apr 22 '24 13:04 pvojtechovsky

@tijsrademakers Is there anything blocking this PR? Any idea when will it be accepted?

edee111 avatar May 28 '24 07:05 edee111

@edee111, it would be good if there were some tests for it as well

filiphr avatar May 28 '24 08:05 filiphr

@edee111, it would be good if there were some tests for it as well

@filiphr I added some tests for it.

tomsvet avatar Jun 04 '24 14:06 tomsvet

@tijsrademakers @filiphr Is there anything else that needs to be added to this PR in order to be accepted?

jkremlacek avatar Jun 11 '24 19:06 jkremlacek

Hi @filiphr , what is the problem with this PR please?

pvojtechovsky avatar Jul 16 '24 11:07 pvojtechovsky

@tijsrademakers @filiphr Hello, when could this MR be merged, please?

tomsvet avatar Aug 12 '24 13:08 tomsvet

It is nearly 6 months since I spent my time to bring feature to the flowable project.

It sounds to me like you are not interested in new features or may be this project is ill? Because having no response from maintainers for nearly 3 months doesn't sound healthy for me.

My trust into flowable project is decreasing. Is flowable project dying? Should I start to search for something more reliable? Could you please give me some inside, so I can understand what is happing here?

Thank you, I hope all is good with you and this PR was just lost. Right?

pvojtechovsky avatar Aug 27 '24 07:08 pvojtechovsky

@pvojtechovsky the Flowable project is not dying.

The reason why this has not been merged yet is because we need to adjust all the not needed formatting changes that have been done as part of this PR.

e.g. there are * imports being used and we do not use in the project, there are also obsolete whitespace changes.

All of that means that we need to spend more work in integrating the PR and thus it takes a bit longer time. I was actually planning to do that this week (even before you wrote)

filiphr avatar Aug 27 '24 08:08 filiphr