Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Parse device brand and model - #28

#28
Merged
wallin merged 2 commits into
ua-parser:masterua-parser/uap-ruby:masterfrom
seandilda:brand_modelseandilda/uap-ruby:brand_modelCopy head branch name to clipboard
Aug 2, 2017
Merged

Parse device brand and model#28
wallin merged 2 commits into
ua-parser:masterua-parser/uap-ruby:masterfrom
seandilda:brand_modelseandilda/uap-ruby:brand_modelCopy head branch name to clipboard

Conversation

@seandilda

Copy link
Copy Markdown
Contributor

This commit adds parsing for the device brand and model.

It also updates to latest uap-core as previously referenced commit is no longer available

Update to latest uap-core as previously referenced commit is no longer available
@wallin

wallin commented May 7, 2017

Copy link
Copy Markdown
Contributor

Thanks @seandilda. This looks good to me. Would you mind rebasing from master to resolve the conflict?

@wallin

wallin commented Jun 13, 2017

Copy link
Copy Markdown
Contributor

Hey @seandilda Do you expect to be able to do the rebase soon? I'd be happy to bring in these changes once done

@wallin

wallin commented Aug 1, 2017

Copy link
Copy Markdown
Contributor

Thanks! @toolmantim any objections?

@toolmantim

Copy link
Copy Markdown
Contributor

None here @wallin! I no longer use the gem, so I'm happy for you to make executive calls 😊

@wallin
wallin merged commit 96ca3e3 into ua-parser:master Aug 2, 2017
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.