-
Notifications
You must be signed in to change notification settings - Fork 212
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
fix Play Store feature filtering #1173
Conversation
<uses-feature android:name="android.hardware.wifi" /> | ||
<uses-feature android:name="android.hardware.location.gps" android:required="true" /> | ||
<uses-feature android:name="android.hardware.telephony" android:required="false" /> | ||
<uses-feature android:name="android.hardware.wifi" android:required="false" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no wifi? I can't imagine we support a no wifi device
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know which devices those might be, but wouldn't it be the same as scanning with wifi disabled? Or is there a problem that will crash the app?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's leave that one out since we are unsure of the implications.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would crash... could be fixed of course in case that is needed at some point.
Updated push to minimum changes.
Looks good! |
@cascheberg fix Play Store feature filtering: telephony not required
@cascheberg is Wi-Fi now required or not? |
wifi is required |
This should fix #1150.
@garvankeeley Just saw that you found the same cause.