← Back to team overview

launchpad-reviewers team mailing list archive

[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