Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

kie-issues#577: Build a first version of the new React-based DMN Editor with ~90% feature parity with the current GWT-based one #2064

Merged
merged 506 commits into from
Dec 24, 2023

Conversation

tiagobento
Copy link
Contributor

@tiagobento tiagobento commented Nov 22, 2023

Closes apache/incubator-kie-issues#577


Good description coming soon...

In the mean time, some issues remain:

1. Properties panel for Boxed Expression cells on Boxed Expression Editor. Moved to apache/incubator-kie-issues#785

2. process is not defined on VS Code extension while trying to import external models. To reproduce, just run dmn-vscode-extension, open a folder with some DMNs on it and click the "Included models" tab, the modal is stuck on "Loading...". There's an error on the Webview Developer Tools.

3. Data Types node panel should display the complete FEEL name of external nodes Data Types. To reproduce, create a new Workspace with a Sample and a new DMN. Import the Sample on the new DMN, and add some external nodes to the default DRD. You can see that the Data Types are red and are not prefixed with the import name you chose.

4. Need to add the Apache headers

tiagobento and others added 30 commits October 9, 2023 14:46
@tiagobento
Copy link
Contributor Author

After a lot of testing done by me and @ljmotta, we're confident on merging this PR when the CI is green!

Copy link
Member

@porcelli porcelli 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 such great addition!

I understand that, due its nature, there was no option to send incremental PRs.

Thank you for all the effort behind this, DMN users will appreciate this and will be even excited to finally see the editor with the same support of the spec as the runtime!

good job!

@tiagobento
Copy link
Contributor Author

@porcelli Thx! This PR is just the beginning. We have a lot of tasks to do!

First we'll finish

and then we'll start continuous improvements to the new DMN Editor!

@tiagobento tiagobento merged commit 665cc86 into apache:main Dec 24, 2023
8 checks passed
@hanjoosten
Copy link

we're

I am really loking forward to the new editor. 🙏💐

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
6 participants