The proposal to merge lp:~wallyworld/launchpad/sharing-details-delete2-966641 into lp:launchpad has been updated.
Description changed to: == Implementation == Add new API to sharing service: def revokeAccessGrants(pillar, sharee, branches=None, bugs=None): """Remove a sharee's access to the specified artifacts. This is used by the delete button on the sharing details page to revoke a user's access to a bug or branch. The findByArtifact and revokeByArtifact methods on IAccessArtifactGrantSource had to be extended to allow an option list of grantees to be passed in, to only find/revoke access for the specified people. I also had to change the sharing details view - it was putting bugtask urls into the view data model but instead needed to use bug urls. Our disclosure data model deals with bugs, not bugtasks. == Tests == Add new revokeAccessGrants tests to test_sharingservice Add new tests for the findByArtifact and revokeByArtifact methods to test_accesspolicy == Lint == Linting changed files: lib/lp/registry/interfaces/accesspolicy.py lib/lp/registry/interfaces/sharingservice.py lib/lp/registry/model/accesspolicy.py lib/lp/registry/services/sharingservice.py lib/lp/registry/services/tests/test_sharingservice.py lib/lp/registry/tests/test_accesspolicy.py For more details, see: https://code.launchpad.net/~wallyworld/launchpad/sharing-details-delete2-966641/+merge/100723 -- https://code.launchpad.net/~wallyworld/launchpad/sharing-details-delete2-966641/+merge/100723 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~wallyworld/launchpad/sharing-details-delete2-966641 into lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
