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

Samsung devices are not identified #494

Closed
koendehondt opened this issue Sep 25, 2020 · 2 comments · Fixed by #496
Closed

Samsung devices are not identified #494

koendehondt opened this issue Sep 25, 2020 · 2 comments · Fixed by #496

Comments

@koendehondt
Copy link

Description

Samsung devices are not identified.

How to reproduce

b = Browser.new("Mozilla/5.0 (Linux; Android 10; SM-G975F) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.101 Mobile Safari/537.36")
b.device.name
=> "Unknown"

What do you expect

b = Browser.new("Mozilla/5.0 (Linux; Android 10; SM-G975F) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/85.0.4183.101 Mobile Safari/537.36")
b.device.name
=> "SM-G975F"

What happened instead

See "How to reproduce"

Software:

  • Rails version: 5.2.4
  • Browser gem version: 5.0.0

Full backtrace

N/A

Report

N/A

@fnando
Copy link
Owner

fnando commented Sep 27, 2020

Would "Samsung Galaxy S10 (SM-G975F)" be better? I can retrieve an initial list of devices, and the community may keep it updated via pull requests. Does that make sense?

fnando added a commit that referenced this issue Sep 27, 2020
fnando added a commit that referenced this issue Sep 27, 2020
@koendehondt
Copy link
Author

@fnando "Samsung Galaxy S10 (SM-G975F)" is fine! Thank you.

fnando added a commit that referenced this issue Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants