Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

KOGITO-9961: Node coloring stopped working on runtime tools #2065

Merged
merged 4 commits into from
Dec 5, 2023

Conversation

handreyrc
Copy link
Contributor

@handreyrc handreyrc commented Nov 22, 2023

Hi @caponetto,

This fix should do the trick. Please, could you take a look.

Basically J2CL has some issues handling "JsTypes".
For now we have to use @GWT3Export along with "JsTypes" to get it working.

JIRA: KOGITO-9961

Thanks

@handreyrc
Copy link
Contributor Author

Hi @caponetto, @LuboTerifaj ,

This PR is ready for review again.
Please, could you take a look?

BTW, Thanks @treblereel for the helping hand.

@caponetto caponetto removed the pr: wip PR is still under development label Nov 29, 2023
Copy link
Contributor

@caponetto caponetto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @handreyrc and @treblereel!

@caponetto caponetto added waiting-for-qe and removed pr: waiting-for-review Waiting for peer reviews labels Nov 30, 2023
Copy link
Contributor

@LuboTerifaj LuboTerifaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the fix @handreyrc , all tests passed and I didn't find any issues.

@caponetto caponetto merged commit 628368a into apache:main Dec 5, 2023
thiagoelg pushed a commit to thiagoelg/kie-tools that referenced this pull request Dec 6, 2023
)

Co-authored-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
rodrigonull pushed a commit to rodrigonull/kie-tools that referenced this pull request Dec 6, 2023
)

Co-authored-by: Dmitrii Tikhomirov <chani.liet@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants