Closed Bug 1030981 Opened 11 years ago Closed 11 years ago

Remove request.amo_user

Categories

(Marketplace Graveyard :: Code Quality, defect, P3)

Product:
Marketplace Graveyard
Component:
Code Quality
Version:
Avenir
Platform:
x86
macOS
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
2014-07-29

People

(Reporter: robhudson, Assigned: mat)

Details

(Whiteboard: [repoman][qa+])

Since the merging of auth_user and users we now have `request.user` and `request.amo_user` which are both the same UserProfile object. Let's get rid of `amo_user` and use `user` everywhere.
Priority: -- → P4
Assignee: nobody → mpillard
Taking and slightly bumping priority because this will really help cleaning up some of our mess regarding auth.
Status: NEW → ASSIGNED
Priority: P4 → P3
https://github.com/mozilla/zamboni/commit/8038e7b8a2b3081640c1232f00fd2092268b8029 QA: - Please test that everything related to authentication and permission still work. I know, it's vague and broad :/ For instance, test logging in and out, pages allowing only authenticated users, etc.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2014-07-29
Whiteboard: [repoman] → [repoman][qa+]
Verified as fixed using more login\logout and authentication permission scenarios
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.