595996 - Discovery pane download source

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Discovery Pane, defect, P2)

Description

[Justin Scott [:fligtar]]

Right now discovery pane links to add-ons with src=api. We should make those links src=discovery-pane and add the new source to the database.

(We don't have to do the pretty name and description yet)

Comment 1

[Jeff Balogh (:jbalogh)]

Aren't these links going to go to special pages?  Does it matter what the src is?

Comment 2

[Jeff Balogh (:jbalogh)]

http://github.com/jbalogh/zamboni/commit/2391a51

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; links are https://preview.addons.mozilla.org/z/en-US/firefox/addon/15679/?src=discovery-pane, etc.

Comment 4

[Justin Scott [:fligtar]]

Did you run the query/migration as well?

(In reply to comment #1)
> Aren't these links going to go to special pages?  Does it matter what the src
> is?

They will eventually go to the special details pages which will use the discovery-pane download source, but until then, we need it here.

Comment 5

[Justin Scott [:fligtar]]

Re-opening for comment #4 question -- did we run the SQL migration?

Also, we should have disco pane sources on the add-ons in the collection promos.

Comment 6

[Jeff Balogh (:jbalogh)]

http://github.com/jbalogh/zamboni/commit/7011f7a

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

All add-ons now have ?src=discovery-pane; not sure about the SQL migration -- someone with access to the DB can check if the stats are working.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #7)
> All add-ons now have ?src=discovery-pane; not sure about the SQL migration --
> someone with access to the DB can check if the stats are working.

From the push bug -- bug 595741:

Run schematic? [y/n]: y
Running migration 77:
INSERT INTO download_sources (name, type, created)
    VALUES ('discovery-pane', 'full', NOW());

Verified FIXED.