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

NO-ISSUE: Sample DMN not compatible with classic DMN Editor #2550

Merged

Conversation

kbowers-ibm
Copy link
Contributor

When opening the sample.dmn file using the classic DMN Editor, it's now incompatible due to the updated dmn:definitions. This PR Reverts them back to the 2018 versions, so that users can interact with the sample.dmn in both the new and classic DMN Editor

Copy link
Contributor

@yesamer yesamer left a comment

Choose a reason for hiding this comment

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

@kbowers-ibm Good catch!

@kbowers-ibm
Copy link
Contributor Author

kbowers-ibm commented Sep 2, 2024

Ubuntu-latest 2 failing due to one test. The test in question is failing in multiple PRs, and passes when run locally

@kbowers-ibm kbowers-ibm changed the title NO-ISSUE: Revert sample dmn definitions NO-ISSUE: Sample DMN not compatible with classic DMN Editor Sep 3, 2024
@ljmotta
Copy link
Contributor

ljmotta commented Sep 3, 2024

@kbowers-ibm great catch. It looks like the branch you used wasn't from main, as it has many commits from other PRs.

@kbowers-ibm
Copy link
Contributor Author

It was originally, but when I did the 3rd commit I messed up the versioning, and couldn't figure out how to revert it. Best I could do was make it so that it only shows the files I changed!

@ljmotta
Copy link
Contributor

ljmotta commented Sep 3, 2024

What I suggest is to rename this branch in your local to another name git branch -m <new_name>, get your commits SHA with git log, create a new branch from main with the same name as the one used in this PR no-issue-rever-sample-dmn-definitions. Apply the commits to this new branch using git cherry-pick <commit_SHA>, and make a push force. With the push force you will fix the commit history, but double/triple check if the commits are correct before you do it.

@kbowers-ibm kbowers-ibm force-pushed the no-issue-rever-sample-dmn-definitions branch from d34183f to 8fc8036 Compare September 4, 2024 06:46
@kbowers-ibm
Copy link
Contributor Author

It worked, thank you very much @ljmotta! I've noted that process down for future reference :)

Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

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

I can confirm this can be opened both classic and new DMN Editor.

@yesamer yesamer merged commit 380d328 into apache:main Sep 5, 2024
14 checks passed
@kbowers-ibm kbowers-ibm deleted the no-issue-rever-sample-dmn-definitions branch September 5, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants