Closed Bug 978770 Opened 10 years ago Closed 10 years ago

/feedback/fxos redirects to /downloadfirefox in browsers that aren't Firefox

Categories

(Input Graveyard :: Submission, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rrosario, Assigned: willkg)

References

Details

(Whiteboard: u=user c=feedback p=1 s=input.2014q2)

If I try to submit feedback about Firefox OS from Chrome, for example, I am told to download Firefox: "Oh! So, you want to offer us feedback on Firefox. Thank you, but you'll need to be on the latest version.".

If I do it from Firefox, I can submit feedback just fine. I can also submit feedback about other Firefox products from Chrome. It's just an issue in the Firefox OS workflow.

This is probably no big deal for now, since we only are requesting feedback on Firefox OS from the devices themselves.
Just to clarify, STR:

1. use Chrome
2. go to /feedback/fxos
3. get greeted with "download firefox" page

That's not ideal since we don't get anything out of forcing firefox os users to use firefox to fill out the feedback form. We should nix the decorator on the view that pushes to the downloadfirefox page.

Frankly, we should ditch that downloadfirefox thing altogether and instead add product-specific notes to the generic form when the user is using a version of that product that's out of date. We could ditch the downloadfirefox thing in this bug, but the product-specific note should be a new bug. Tyler has wanted that for a while, so it'd be good to do, but it requires some thought.
We want more Firefox OS feedback, so we probably want to nix this as soon as we can.

I'm making this a bite-sized bug.

The gist of it is that we should delete the "requires_firefox" function in fjord/feedback/views.py and fix any place it's being used and remove the relevant tests.

If you're interested in working on this bug, let us know in #input on irc.mozilla.org and/or comment on this bug.
Whiteboard: u=user c=feedback p= s=input.2014q1 [mentor=willkg]
Moving this to 2014q2.
Priority: -- → P1
Whiteboard: u=user c=feedback p= s=input.2014q1 [mentor=willkg] → u=user c=feedback p= s=input.2014q2 [mentor=willkg]
Grabbing this and estimating it at 1 point.
Assignee: nobody → willkg
Whiteboard: u=user c=feedback p= s=input.2014q2 [mentor=willkg] → u=user c=feedback p=1 s=input.2014q2
Landed in master: https://github.com/mozilla/fjord/commit/5ae05bb

Will push on Tuesday.
Pushed to prod just now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Product: Input → Input Graveyard
You need to log in before you can comment on or make changes to this bug.