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

Added touch_id method #445

Merged
merged 1 commit into from
Dec 13, 2016
Merged

Added touch_id method #445

merged 1 commit into from
Dec 13, 2016

Conversation

dpgraham
Copy link
Contributor

@dpgraham dpgraham commented Nov 9, 2016

-Wrote the touchId method (see http://stackoverflow.com/questions/25459059/is-there-any-way-to-use-apples-touch-id-fingerprint-scanner-on-ios-simulator for info on touchID simulator)
-Added API documentations to jsonwire-full-mapping.md, api.md and jsonwire-full.json.
-Wrote a test in msjon-specs.js.

Wrote the method, added API documentations to jsonwire-full-mapping.md, api.md and jsonwire-full.json. Wrote a test in msjon-specs.js.
@dpgraham
Copy link
Contributor Author

dpgraham commented Nov 9, 2016

@jlipps or @imurchie When either of you have a chance can you look at the travis ci logs. It's a Sauce Connect error.

@imurchie
Copy link
Collaborator

imurchie commented Nov 9, 2016

@dang: This has to do with the access PRs have to the encrypted environment variables used to connect through SC. It's a security risk to allow them to be accessed within PRs (since random people would then have them).

You can see that all the PRs in this repo fail. :(

@dang
Copy link

dang commented Nov 9, 2016

@imurchie I think you tagged the wrong account. You probably meant @dpgraham

@dpgraham
Copy link
Contributor Author

dpgraham commented Nov 9, 2016

Are we able still able to merge this request?

@imurchie
Copy link
Collaborator

Whoops! Used his Slack handle.

@dpgraham I think we ought to be able to. I'm not an owner of this repo, though.

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.

4 participants