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

Adds detection for Spark, Total Browser, improves detection for Belva Browser, Phoenix and rename MIUI Browser to Mi Browser #7741

Merged
merged 15 commits into from
Jul 31, 2024

Conversation

liviuconcioiu
Copy link
Collaborator

@liviuconcioiu liviuconcioiu changed the title Adds detection for Spark, Total Browser and rename MIUI Browser to Mi Browser Adds detection for Spark, Total Browser, improves detection for Belva Browser and rename MIUI Browser to Mi Browser Jul 14, 2024
@liviuconcioiu liviuconcioiu changed the title Adds detection for Spark, Total Browser, improves detection for Belva Browser and rename MIUI Browser to Mi Browser Adds detection for Spark, Total Browser, improves detection for Belva Browser, Phoenix and rename MIUI Browser to Mi Browser Jul 18, 2024
@@ -570,6 +570,7 @@ class Browser extends AbstractClientParser
'LE' => 'Smart Lenovo Browser',
'OZ' => 'Smooz',
'SN' => 'Snowshoe',
'K5' => 'Spark',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Googling seems to suggest that there have been only "Baidu Spark" browser (which we do have already). Is this the one or is it something different?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -393,7 +393,7 @@ class Browser extends AbstractClientParser
'M3' => 'Midori Lite',
'M6' => 'MixerBox AI',
'MO' => 'Mobicip',
'MU' => 'MIUI Browser',
'MU' => 'Mi Browser',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sgiehl Do you agree to change the name?

@sanchezzzhak sanchezzzhak merged commit 6834e0f into matomo-org:master Jul 31, 2024
15 checks passed
@liviuconcioiu liviuconcioiu deleted the browsers1 branch August 1, 2024 04:51
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 this pull request may close these issues.

3 participants