Get PersonaAuthor model or substitute

RESOLVED FIXED in 6.4.8

Status

addons.mozilla.org Graveyard
Code Quality
RESOLVED FIXED
6 years ago
2 years ago

People

(Reporter: Andy McKay, Assigned: Andy McKay)

Tracking

unspecified
6.4.8

Details

(Assignee)

Description

6 years ago
In bug 726186 I naively made the PersonAuthor a real Django model. This then uncovered the problems in bug 734256. Apparently we aren't ready for just flipping over PersonaAuthor to being a real user object. 

Quite a lot of things have to be cleaned up, including but not limited to: the Addon transformer, the fact that persona tests assume and create a dummy user, the templates will just try and render all the stuff off the PersonaAuthor as if its just a user.

So I reverted 726186 apart from the migration.

What we've got to do is pull "make the PersonaAuthor a Django model" part out of that bug. Clean up. Write some tests, fix the above.

Then worry about linking up the migration.
(Assignee)

Comment 1

6 years ago
Should be bug 734526.
(Assignee)

Comment 2

6 years ago
I'm not sure what this model is for. Let's remove it and the references as noted above. Instead let's just use the standard AddonUser. Once that's working as expected, bug 726186 can land.
(Assignee)

Comment 3

6 years ago
https://github.com/mozilla/zamboni/commit/fd3ad4
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Assignee: nobody → amckay
Target Milestone: --- → 6.4.6
Target Milestone: 6.4.6 → 6.4.8
(Assignee)

Comment 5

6 years ago
https://github.com/mozilla/zamboni/commit/efc0e8

If this is still a problem got lots of time to fix and avoid reverting :)
Status: REOPENED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.