Merge lp://staging/~mardy/click-reviewers-tools/new-account-hook into lp://staging/click-reviewers-tools
Status: | Merged |
---|---|
Merged at revision: | 516 |
Proposed branch: | lp://staging/~mardy/click-reviewers-tools/new-account-hook |
Merge into: | lp://staging/click-reviewers-tools |
Diff against target: |
538 lines (+367/-8) 4 files modified
clickreviews/cr_common.py (+2/-0) clickreviews/cr_online_accounts.py (+159/-4) clickreviews/cr_tests.py (+26/-4) clickreviews/tests/test_cr_online_accounts.py (+180/-0) |
To merge this branch: | bzr merge lp://staging/~mardy/click-reviewers-tools/new-account-hook |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jamie Strandboge (community) | Approve | ||
Review via email:
|
Commit message
Support the new "accounts" hook
A new hook for Online Accounts is being added, and it's intended to replace both the "account-
With this change we check that the new "accounts" hook becomes the mandatory OA hook starting from the 15.10 framework.
Description of the change
Support the new "accounts" hook
A new hook for Online Accounts is being added, and it's intended to replace both the "account-
With this change we check that the new "accounts" hook becomes the mandatory OA hook starting from the 15.10 framework.
Thanks for the MP! I've got a few comments inline.