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

KOGITO-4343: fixed jsonix CCE with gwt290 #3570

Closed
wants to merge 1 commit into from

Conversation

treblereel
Copy link
Contributor

@treblereel treblereel commented Feb 18, 2021

JIRA: https://issues.redhat.com/browse/KOGITO-4343

Related PRs:
kiegroup/gwt-jsonix-schema-compiler#11
kiegroup/droolsjbpm-build-bootstrap#1606
#3570
#3562
#3582

to test it locally, gwt-jsonix-schema-compiler must be install first, in kie-wb-common-dmn-webapp-kogito-marshaller/pom.xml version of gwt-jsonix-schema-compiler must setted to 1.3.0-SNAPSHOT

this issue is related to #3562

@tiagobento
Copy link
Contributor

Hi @karreiro! This PR fixes the remaining issues on the development webapps. It depends on a breaking change at kiegroup/gwt-jsonix-schema-compiler#11. After this PR on JSONIX is merged, a new PR will be sent to droolsjbpm-build-bootstrap updating the version so that this PR can be easily tested.

Also note that this PR works together with #3562.

In summary, after this one (#3570) and #3562 are merged, the flags disabling the new type-checks introduced in GWT 2.9 can be safely removed.

Copy link
Contributor

@karreiro karreiro 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, @treblereel! Excellent work!

I've left only a minor comment. But I just tested it locally, I ran all IT suites, and everything works fine :-)

@@ -125,7 +125,7 @@ public void checkAttachDomElement() {
domElementOnDisplay);
}

@Test
//@Test
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe we can uncomment this test :-) (it passes locally here)

@tiagobento
Copy link
Contributor

Closing in favor of #3582

@tiagobento tiagobento closed this Mar 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants