Merge lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators into lp://staging/postorius
Proposed by
Bhavesh Goyal
Status: | Merged |
---|---|
Merged at revision: | 221 |
Proposed branch: | lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators |
Merge into: | lp://staging/postorius |
Diff against target: |
38 lines (+9/-4) 2 files modified
src/postorius/auth/decorators.py (+8/-3) src/postorius/templates/postorius/menu/list_nav.html (+1/-1) |
To merge this branch: | bzr merge lp://staging/~bhavesh-goyal093/postorius/fixed-postorius-method-decorators |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Terri | Approve | ||
Review via email: mp+256037@code.staging.launchpad.net |
Description of the change
Fixed Bug #1435062 which caused postorius to show incorrect nav bar view for members posessing different roles.
The solution was found in correcting permissions for list_nav template and fixing method decorators for list_moderator_
Now the attributes are set correctly according to the roles and the nav bar sows correct view or different and multiple roles.
To post a comment you must log in.
Whoops! Looks like you used tabs instead of spaces. I'll fix it when I do the merge, assuming my tests pass.