Closed Bug 519861 Opened 15 years ago Closed 15 years ago

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

Categories

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

Product:
addons.mozilla.org Graveyard
Component:
Public Pages
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: clouserw, Assigned: jbalogh)

References

(
URL
)

Details

Attachments

(3 files, 4 obsolete files)

personas import script
19.83 KB, patch
clouserw
: review+
Details | Diff | Splinter Review
using addon_categories, adding ADDON_PERSONA to amo
7.40 KB, patch
clouserw
: review+
Details | Diff | Splinter Review
don't try to add NULL categories, mysql timeout fix
1.80 KB, patch
clouserw
: review+
Details | Diff | Splinter Review
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.
Priority: -- → P1
Depends on: 519844
Attached patch personas import script (obsolete) — — Splinter Review
Attachment #404860 - Flags: review?(clouserw)
Attached patch personas import script (obsolete) — — Splinter Review
now with a lockfile and personas disabling
Attachment #404952 - Flags: review?(clouserw)
Attachment #404860 - Attachment is obsolete: true
Attachment #404860 - Flags: review?(clouserw)
Depends on: 520924, 520681
Attached patch personas import script (obsolete) — — Splinter Review
importing approval times, fixing a couple bugs
Attachment #405158 - Flags: review?(clouserw)
Attachment #404952 - Attachment is obsolete: true
Attachment #404952 - Flags: review?(clouserw)
Attachment #405160 - Flags: review?(clouserw)
Attachment #405158 - Attachment is obsolete: true
Attachment #405158 - Flags: review?(clouserw)
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 on attachment 405160 [details] [diff] [review] personas import script Why not use the existing categories table?
Attachment #405160 - Flags: review?(clouserw) → review+
incremental patch so you can see what's changing
Attachment #405295 - Flags: review?(clouserw)
Using addon_categories is great, that'll make my life in bug 519863 much happier.
one more incremental fix
Attachment #405325 - Flags: review?(clouserw)
Attachment #405295 - Flags: review?(clouserw) → review+
Attachment #405325 - Flags: review?(clouserw) → review+
Added in r53106 and r53107.
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: push-needed
Resolution: --- → FIXED
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.
(In reply to comment #11) > I'll add some default values to stop the pain. Pain relievers committed to r53132.
Blocks: 521323
Attached patch personas list view (obsolete) — — Splinter Review
with all the goodies this time
Attachment #405518 - Flags: review?(clouserw)
Attachment #405518 - Attachment is obsolete: true
Attachment #405518 - Flags: review?(clouserw)
Comment on attachment 405518 [details] [diff] [review] personas list view oops, wrong bug
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: