-
Notifications
You must be signed in to change notification settings - Fork 165
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
Replaced generic phone model by device model #253
Conversation
For android devices we see the specific device model in the backend, for iOS devices we only see generic "iPad" or "iPhone". Here we replace the generic string in the user agent with the real device model (e.g. iPhone8,4), which lets the backend properly display "iPhone SE".
Hi @Pe-te, thanks a lot for this PR. This isn't a method that came in mind, but it's a great idea! |
I love it! Looks great! Can you do me a favor and add an entry to the CHANGELOG.md? Feel free to reference yourself there. |
Thanks a lot! I will wait for the tests to finish (just to make sure everything works well 😊) and will merge. |
Thx! If it ever causes trouble, maybe there's a cleaner way, where you add the model as extra parameter in the brackets, but this works for us right now. |
…-org#253, which was removed in matomo-org#308
…-org#253, which was removed in matomo-org#308
…-org#253, which was removed in matomo-org#308
…ture matomo-org#253, which was removed in matomo-org#308
For android devices we see the specific device model in the backend, for iOS devices we only see generic "iPad" or "iPhone".
Here we replace the generic string in the user agent with the real device model (e.g. iPhone8,4), which lets the backend properly display "iPhone SE".