519861 - Write a script to synchronize getpersonas.com and AMO add-ons

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P1)

Description

[Wil Clouser [:clouserw]]

This is 1.3 in the spec.  Any new personas that are approved on getpersonas should be imported into the AMO db via cron.  I've filed bug 519844 to get us personas db dumps and there is a dev site at sm-personas01.mozilla.org if you want it.  

I can help you with details here.  We don't need preview images or anything since we'll be loading those directly off getpersonas.com.  Also, in my head this works, we should just associate all the imports with a generic user account for personas.  When authors are "imported" we're just pulling in their names.  Any clicks on the authors name will go to getpersonas.com.

Comment 1

[Jeff Balogh (:jbalogh)]

Attachment: personas import script

Comment 2

[Jeff Balogh (:jbalogh)]

Attachment: personas import script

now with a lockfile and personas disabling

Comment 3

[Jeff Balogh (:jbalogh)]

Attachment: personas import script

importing approval times, fixing a couple bugs

Comment 4

[Jeff Balogh (:jbalogh)]

Attachment: personas import script

Comment 5

[Fred Wenzel [:wenzel]]

You probably want to add the new add-on type to config/constants.php as well as models/addontype as well. Otherwise it won't show up in a few places, like the admin category manager.

Comment 6

[Wil Clouser [:clouserw]]

Comment on attachment 405160 [details] [diff] [review]
personas import script

Why not use the existing categories table?

Comment 7

[Jeff Balogh (:jbalogh)]

Attachment: using addon_categories, adding ADDON_PERSONA to amo

incremental patch so you can see what's changing

Comment 8

[Fred Wenzel [:wenzel]]

Using addon_categories is great, that'll make my life in bug 519863 much happier.

Comment 9

[Jeff Balogh (:jbalogh)]

Attachment: don't try to add NULL categories, mysql timeout fix

one more incremental fix

Comment 10

[Jeff Balogh (:jbalogh)]

Added in r53106 and r53107.

Comment 11

[Fred Wenzel [:wenzel]]

preview.AMO is flooding the amo-developers mailing list with warnings about PERSONAS_DB_HOST undefined etc...

I'll add some default values to stop the pain.

Comment 12

[Fred Wenzel [:wenzel]]

(In reply to comment #11)
> I'll add some default values to stop the pain.

Pain relievers committed to r53132.

Comment 13

[Jeff Balogh (:jbalogh)]

Attachment: personas list view

with all the goodies this time

Comment 14

[Jeff Balogh (:jbalogh)]

Comment on attachment 405518 [details] [diff] [review]
personas list view

oops, wrong bug