Sniff for mobile devices on submission page load

VERIFIED FIXED in 3.0

Status

Input
General
P1
enhancement
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: aakashd, Assigned: wenzel)

Tracking

Details

(Reporter)

Description

7 years ago
Steps to reproduce:
1. Go to input.mozilla.com/happy or /sad from fennec.
2. Go to input.stage.mozilla.com/happy or /sad from fennec.

Actual Results:
Desktop submission page is loaded.

Expected Results:
Mobile submission pages should be loaded.
(Reporter)

Updated

7 years ago
Priority: -- → P1
Target Milestone: --- → 2.1
(Assignee)

Comment 1

7 years ago
Are they supposed to do that? That's a new feature, which we aren't supposed to add during code freeze.

It's pretty simple though, so I'll probably take a look on Monday.
Severity: major → enhancement
Summary: staging and prod are not sniffing for mobile devices on submission page load → Sniff for mobile devices on submission page load
(Assignee)

Comment 2

7 years ago
I'll need to mess with the way we detect Firefox mobile a bit for this, and I'd rather have it thoroughly tested before we go live, so I can't do it during this code freeze. Punting to 3.0.
Target Milestone: 2.1 → 3.0
(Reporter)

Comment 3

7 years ago
This is actually a regression as clicking on either of the happy/sad buttons in the client will show up the desktop submission pages instead.
(Assignee)

Comment 4

7 years ago
We discussed this in person shortly, and while it is a moot point if this is a regression or not, I am wondering where the bug came from. It seems to me that the extension is not pointing to m.input.moz.com anymore, but just to input.moz.com now. Do we need to inform the Mobile team of this?
(Reporter)

Comment 5

7 years ago
Aye; I think that's the best option going forward. Unfortunately, they're about to release, so its pretty much impossible to change it for beta 3.
(Assignee)

Updated

7 years ago
Depends on: 621088
(Assignee)

Comment 6

7 years ago
I am forwarding known mobile devices to the same path on the mobile domain now. The URLs covered for now (subject to change in bug 621088) are the dashboard as well as the feedback pages.

http://github.com/fwenzel/reporter/commit/3835b47

This contains unit tests.
Assignee: nobody → fwenzel
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Verified fennec redirected to m.input.stage, and displayed the correct happy/sad mobile page
Status: RESOLVED → VERIFIED
Component: Input → General
Product: Webtools → Input
You need to log in before you can comment on or make changes to this bug.