launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #33117
[Merge] ~enriqueesanchz/launchpad:fix-update-cve into launchpad:master
The proposal to merge ~enriqueesanchz/launchpad:fix-update-cve into launchpad:master has been updated.
Description changed to:
When giving permissions to modify a JSONB/dict field, you don't get permissions to modify existing keys in place. You need to change the entire `cve.metadata = new_value` instead of doing `cve.metadata['affected'] = new_value`.
This error was hidden in the tests.
For more details, see:
https://code.launchpad.net/~enriqueesanchz/launchpad/+git/launchpad/+merge/494096
--
Your team Launchpad code reviewers is requested to review the proposed merge of ~enriqueesanchz/launchpad:fix-update-cve into launchpad:master.
References