Skip to content
This repository was archived by the owner on Sep 28, 2024. It is now read-only.

Conversation

@aosmond
Copy link

@aosmond aosmond commented Dec 15, 2014

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

could it be replaced with var usePromise = !(version && version.length === 2 && version[1] < 37) ? that way it should work on other browsers if they decided to implement the last version of the API

@mathroc
Copy link
Member

mathroc commented Dec 23, 2014

Hi!

thank you for this PR :)
I don't have a device I can test this on. did you tested it on 2.0 and/or 2.1. I added 2 comments on your code can you take a look at it?

thx!

@mathroc
Copy link
Member

mathroc commented Dec 26, 2014

OS 2.2 compatibility has been fixed in 9ed896e

@mathroc mathroc closed this Dec 26, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants