-
Notifications
You must be signed in to change notification settings - Fork 344
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
Temporary fix to metadata editor display bug #2593
Conversation
Thanks for making a pull request to Elyra! To try out this branch on binder, follow this link: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, also wanted to note that this doesn't fix the bug noted in #2590 so that should still be done at some point
@marthacryan as this is a temporary fix, it shouldn't |
@lresende Oh I see what you're saying but it's fixing that bug / issue. It's temporary in that we want to add back in |
This is actually rolling back a "feature addition" that caused the bug seen in that issue. So it does fix the issue by removing the new unreleased feature. We will then reintroduce the feature along side the new form later on. The feature is pretty minor and was a tiny "nice to have" that I added when I noticed how "easy" it was (which apparently was wrong) |
Fixes #2592.
Developer's Certificate of Origin 1.1