Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android users visiting /firefox/ pages get enabled Install buttons #12293

Closed
jvillalobos opened this issue Sep 4, 2018 · 9 comments · Fixed by mozilla/addons-frontend#8897
Closed
Assignees
Labels
Milestone

Comments

@jvillalobos
Copy link

This might be a duplicate, but I couldn't find it. It came up again in this Discourse thread.

The problem is that Android users may end up on a /firefox/ link on AMO because that's how it's generally linked to online. When a /firefox/ page is visited on Android, the Install button is enabled even if the add-on is not compatible.

Maybe we should redirect Android users to /android/, or make the buttons check for device compatibility regardless of the platform in the URL, or both.

@muffinresearch
Copy link
Contributor

@jvillalobos The button being enabled when incompatible definitely sounds like a bug that we should fix.

For the other part, we generally don't second guess users based on URL parameters. Whether that's client application or locale.

One way we could deal with this is to set a banner to inform the user they're viewing a page that doesn't match their platform. They would then have the option to follow a link to see the same page for the correct client application or optionally we could also add a cookied "don't show this again" to not ever show that banner again whilst the cookie is present.

@jvillalobos
Copy link
Author

Fixing the install button and adding the banner sounds like a good approach to me.

@stale
Copy link

stale bot commented Oct 7, 2019

This issue has been automatically marked as stale because it has not had recent activity. If you think this bug should stay open, please comment on the issue with further details. Thank you for your contributions.

@stale stale bot added the state:stale Issues marked as stale. These can be re-opened should there be plans to fix them. label Oct 7, 2019
@sneakypete81
Copy link

This is still an issue: https://addons.mozilla.org/en-GB/firefox/addon/update-scanner/ shows an install button on Android, despite the add-on being marked as incompatible.

@stale stale bot removed the state:stale Issues marked as stale. These can be re-opened should there be plans to fix them. label Oct 7, 2019
@jvillalobos
Copy link
Author

Just added this to a Q4 project related to this problem. It's a secondary goal, so it's okay to drop if there isn't enough time to cover it.

@sneakypete81
Copy link

sneakypete81 commented Nov 12, 2019

@bobsilverberg thanks for looking at this. Unfortunately the recent changes haven't fixed this issue. I now see the yellow warning when browsing the amo/firefox section, but the "Add to Firefox" button is still enabled. The install button should always be disabled on the Android platform if the addon is not compatible with Android.

STR:

Expected behaviour:

  • "Add to firefox" button disabled and warning displayed.

Actual behaviour:

  • "Add to firefox" button enabled and warning displayed ("This listing is not intended for this platform. Browse add-ons for Firefox on Android")

@bobsilverberg
Copy link
Contributor

@sneakypete81 This fix has not been deployed to production yet. The fix should be there after the push on Thursday, November 14th, which happens around noon EST.

@sneakypete81
Copy link

Ah, ok. Thanks, I'll be patient!

@ioanarusiczki
Copy link

@bobsilverberg Verified on dev with FF68.2.0 (Android 8.0)
The button is disabled.

the button is inactive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants