-
Notifications
You must be signed in to change notification settings - Fork 335
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
Update metadata API to require either name or display_name #629
Conversation
Adds behavior of reopening the code snippets extension on reloading jupyter lab (if code snippets was already open). Fixes elyra-ai#432
Fixes elyra-ai#550 Co-authored-by: Alex Bozarth <ajbozart@us.ibm.com>
Update metadata API accept missing name if the display_name is provided addressing elyra-ai#623
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.
Thanks Va - just a couple comments.
Regarding the swagger doc update, I guess I'm not following how removing 'name' as required from MetadataResourceBodyPost
applies to the issue you list regarding query parameters. Once we support query parameters, I would expect them to apply to the various GET handlers anyway and I figured only the body for POST would require an update here. I'm sorry if I'm not understanding the issue.
cleam up comment based on review comments
@kevin-bates my apologies, the misunderstanding was on my part. not sure how or why i was confusing the query parameters with the request body. you can ignore that comment! |
No worries at all Va! |
Hi Va - your particular branch is causing build errors. Not sure if its due to how you maintain your branches or not but I can't reproduce the build issue unless I pull and build your branch. My system uses an older version of yarn than what GH Actions are using, but the results are the same...
|
thank you @kevin-bates, i noticed that and am looking into it. the failure is happening with the there was a new release of i will continue to look into this further to make sure i am not missing anything, but i suspect you should see the same error in any new/clean build which ends up downloading the latest |
I see. Thank you for the explanation. I suppose this is just a way of life Dependency-land! Not quite the happiest place on earth. |
Configure selective version resolution for @blueprint components
i just pushed a change that should pin the |
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.
Thanks Va - this is a nice enhancement for display names.
Does the update of the build-break change now need to be backed out of this PR or will it just get subsumed by git's conflict resolution stuff?
it should be resolved by git but i will merge the update and confirm. if necessary will back it out |
Co-authored-by: Karla Spuldaro <karla.spuldaro@ibm.com>
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
Update the
/metadata/<namespace>
POST endpoint to allow missingname
field if thedisplay_name
is providedFixes #623
Fixes #632
FYI, the swagger docs continue to still indicatename
is required. I did not update it because at the moment swagger does not provide a mechanism for specifying any one of a set of parameters is required.Developer's Certificate of Origin 1.1