572913 - Searches should have feeds

Status: VERIFIED FIXED

(Input :: General, enhancement, P1)

Description

[Jesse Ruderman]

I'd like to be able to subscribe to a feed that shows me when feedback mentions "security" or my web site.

Comment 1

[Aakash Desai [:aakashd]]

This is a great idea, here's some info about it even though you probably don't need it.

http://docs.djangoproject.com/en/dev/ref/contrib/syndication/

Comment 2

[Fred Wenzel [:wenzel]]

Probably pretty easy to implement, but first we need search to work.

Comment 3

[Aakash Desai [:aakashd]]

Attachment: Mcok Ups for Dashboard and Search Results Pages

Mcok Ups attached!

Comment 4

[Fred Wenzel [:wenzel]]

Why do we need a huge button? Isn't the feed symbol in the URL bar sufficient? Hmmm.

Comment 5

[Aakash Desai [:aakashd]]

Gives people an idea that it's there!

Comment 6

[Aakash Desai [:aakashd]]

I'm ok with not shipping this if we don't have the time to implement this for a Monday night or Tuesday morning push.

Comment 7

[Fred Wenzel [:wenzel]]

Yes, that's definitely not going to be part of today's push.

Comment 8

[Aakash Desai [:aakashd]]

Alrighty, moving this to 1.5

Comment 9

[Aakash Desai [:aakashd]]

To reduce load, we should only poll for the latest message received every 30 seconds. If that's still too big of a load, continue to double the interval polled until the load is non-existent.

Comment 10

[Aakash Desai [:aakashd]]

Comment on attachment 455694 [details]
Mcok Ups for Dashboard and Search Results Pages

Let's forego the links on the pages themselves. All we need here is the ability to add a feed to our search results page (save dev time!) and there's enough muscle memory to know to look for a feed button if we communicate it out well enough.

Comment 11

[Fred Wenzel [:wenzel]]

Tentatively assigning to Dave Dash, who might help us out with this after the next AMO milestone.

Comment 12

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

http://github.com/fwenzel/reporter/commits/2280a5f

Comment 13

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

1) The feed doesn't validate: http://feedvalidator.org/check.cgi?url=http%3A%2F%2Finput.stage.mozilla.com%2Fen-US%2Fsearch%2Fatom%2F%3Fq%3Dcrashes%26product%3Dfirefox
2) Links are to example.com

Comment 14

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

Oh, and:

3) http://input.stage.mozilla.com/en-US/search/atom/?q=Hi+%26+bye!&product=firefox the ampersand isn't getting unescaped properly in the feed.

Comment 15

[Aakash Desai [:aakashd]]

Dave's not worried about the bugs mentioned blocking the release. Can we re-file these issues on a separate bug and close this one?

Comment 16

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

(In reply to comment #15)
> Dave's not worried about the bugs mentioned blocking the release. Can we
> re-file these issues on a separate bug and close this one?

Sure; close this, and I'll re-file those.

Comment 17

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

Verified FIXED; filed:

bug 580423, bug 580424, and bug 580426.

Comment 18

[Jesse Ruderman]

And I filed bug 580465 against Firefox for not showing a feed preview.