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

Add HipChat, fix Slack, and a few browsers on Android. #245

Merged
merged 1 commit into from
May 24, 2017

Conversation

ojaber
Copy link
Contributor

@ojaber ojaber commented May 12, 2017

Add support for HipChat on Windows and Mac.
Fix support for Slack on Mac.
Fix Maxthon and Crosswalk browsers which used to get recognized as Chrome Mobile on Android.

@commenthol
Copy link
Contributor

@ojaber Thanks for your PR. Maybe by accident you deleted a previous testcase.

  - user_agent_string: 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/601.7.8 (KHTML, like Gecko) Slack_SSB/2.0.3'
    family: 'Slack Desktop Client'
    major: '2'
    minor: '0'
    patch: '3'

Can you please add this in again? Rebasing and squashing your changes to a single commit would be very nice. Thx.

Fix Slack Desktop Client detection on Windows and Mac.
Add support for HipChat on Mac and Windows.
Fix Macthon and Crosswalk support on Android.
@ojaber
Copy link
Contributor Author

ojaber commented May 22, 2017

@commenthol Thanks for checking out the pull request :)
Updated as requested.

@commenthol commenthol merged commit 63f5c66 into ua-parser:master May 24, 2017
@commenthol
Copy link
Contributor

Thx

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.

2 participants