Merge lp://staging/~ricardokirkner/rnr-server/fix-acl-verify-request into lp://staging/rnr-server

Proposed by Ricardo Kirkner
Status: Merged
Approved by: Ricardo Kirkner
Approved revision: 321
Merged at revision: 320
Proposed branch: lp://staging/~ricardokirkner/rnr-server/fix-acl-verify-request
Merge into: lp://staging/rnr-server
Diff against target: 99 lines (+25/-4)
3 files modified
django_project/settings_base.py (+1/-1)
src/core/tests/test_utilities.py (+20/-0)
src/core/utilities.py (+4/-3)
To merge this branch: bzr merge lp://staging/~ricardokirkner/rnr-server/fix-acl-verify-request
Reviewer Review Type Date Requested Status
Facundo Batista (community) Approve
Matt Goodall (community) Approve
Review via email: mp+295721@code.staging.launchpad.net

Commit message

ensure acl verify request is made properly

send body data as a json encoded dict

To post a comment you must log in.
Revision history for this message
Facundo Batista (facundo) :
review: Needs Information
Revision history for this message
Matt Goodall (matt-goodall) wrote :

lgtm, with a question

review: Approve
Revision history for this message
Ricardo Kirkner (ricardokirkner) :
Revision history for this message
Facundo Batista (facundo) wrote :

Thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches