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

Save mindfulness session #14

Merged
merged 8 commits into from
Sep 17, 2017
Merged

Save mindfulness session #14

merged 8 commits into from
Sep 17, 2017

Conversation

skleest
Copy link
Contributor

@skleest skleest commented Jan 20, 2017

Saves mindfulness session

@skleest skleest mentioned this pull request Jan 20, 2017
@terrillo
Copy link
Collaborator

@skleest can you update the readme as well?

@paulief
Copy link

paulief commented Apr 13, 2017

@terrillo I was just wondering if you think this'll be merged soon.

Also, just a related question for @skleest : Do I need to target iOS 10 only to use this? Or is there a way to conditionally include the Mindfulness part of the library on older SDKs?

Thanks guys for all your work on this!

@simonstead
Copy link

Yeah I'd love to know the progress on this too 😀

@EnricoGallus
Copy link

EnricoGallus commented May 12, 2017

+1

@seanadkinson
Copy link

seanadkinson commented May 23, 2017

@skleest I've published a branch that can read Mindful Minutes (maybe I should rename to Mindful Session?). The repo is here: https://github.com/ProvataHealth/react-native-apple-healthkit

It's available on npm with yarn add react-native-apple-healthkit-rn0.40@0.3.2

If you are interested in creating a PR with the ability to write mindful minutes, I'll merge and publish from that repo. Otherwise I'll probably get to it once we need that ability in our app.

@skleest
Copy link
Contributor Author

skleest commented Jun 5, 2017

Hi @paulief, I've been using it for only iOS 10+ using JS (check for device info), and it hasn't been a problem for iOS <10. @seanadkinson this PR includes reading as well!

Currently I'm using RN 0.39 so I'm refraining from updating this repo; if anyone else can fix the small merge conflict, should be good to go.

@terrillo terrillo merged commit 5adade4 into GregWilson:master Sep 17, 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.

6 participants