-
Notifications
You must be signed in to change notification settings - Fork 103
Issue 6276 - Schema lib389 object is not keeping custom schema data upon editing #6279
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
Conversation
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
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.
It definitively looks nicer with the parametrized fixture ! -:)
LGTM for me but you should better wait for the tests completion and check the result before merging.
…pon editing Description: Fix the condition in _edit_schema_object function, so it corretcly evaluates when we need to set an empty value to schema object. Add tests to cover add and edit Schema operations. Fixes: 389ds#6276 Reviewed by: ?
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
…pon editing (#6279) Description: Fix the condition in the _edit_schema_object function to correctly evaluate when to set an empty value to the schema object. Add tests to cover adding and editing Schema operations. Fixes: #6276 Reviewed by: @tbordaz, @progier389 (Thanks!)
Description: Fix the condition in _edit_schema_object function, so it corretcly evaluates when we need to set an empty value to schema object. Add tests to cover add and edit Schema operations.
Fixes: #6276
Reviewed by: ?