search

apache / incubator-kie-issues

Apache License 2.0
12 stars 1 forks source link

On KIE Sandbox it's not possible to use the import DMN on an Business Rule node after renaming the file #530

Open ljmotta opened 1 year ago

ljmotta commented 1 year ago

The KIE Sandbox FS hangs if you try to import a DMN in a BPMN that was renamed after the node was created. All the following edits will not be saved, but If you delete the Business Rule node, the FS will be recovered.

Steps to reproduce:

video

https://github.com/kiegroup/kie-issues/assets/24302289/e5aac8fb-e1bc-47b4-b681-bfdfd1631e03

### Tasks
- [ ] It should be possible to import an DMN after the file is renamed.
inodeman commented 1 year ago

Hi @tiagobento checking on it. Will let you know an ETA

eduardocerqueira commented 1 year ago

for planning purpose, @inodeman can you pls share an update on this? how close are you to send a PR and when could we expect for it?

inodeman commented 1 year ago

As you are well aware @eduardocerqueira been in medical leave for the last weeks so I have not looked at it closely, also if I recall correctly you told me not to focus on GWT issues that you had reassigned it. Let me know if that changed

eduardocerqueira commented 1 year ago

nothing changed, this issue was brought up to my attention during a meeting today and I was asking for an update as I noticed it was still assigned to you. Just unassigned it, we will add it back to the plan in the next planning cycle.

jomarko commented 2 months ago

I was not able to reproduce this issue as of 16of July 2024 using https://sandbox.kie.org/dev. Not sure if there is a step I do differently than the reporter did.

ljmotta commented 2 months ago

@jomarko I still get the error. Did you renamed the file? This step is crucial.

jomarko commented 2 months ago

I renamed the BPMN file, should I rename DMN file? I think that is not clear from the ticket description.

ljmotta commented 2 months ago

No, just the BPMN file. But you must rename the file after creating the node.

inodeman commented 2 months ago

This should do it On 29 Jul 2024, at 8:00 a.m., Luiz João Motta @.***> wrote: No, just the BPMN file. But you must rename the file after creating the node.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>