rn-apple-healthkit icon indicating copy to clipboard operation
rn-apple-healthkit copied to clipboard

Is this project abandonned?

Open olegfeferman opened this issue 3 years ago • 25 comments

Hi everyone,

I see that last release was half a year ago. We wanted to use rn-apple-healthkit in our application but there are few fixes that needs to be done. We thought to fixt it and then submit a PR but then I've seen that there are 32 unmerged PRs (we even found fixes for some of the issues we're having among them). I tried to reach @terrillo but with no luck. Are there other contributors who can potentially review and merge PRs and release an updated version? What is the right way to proceed? Is there a way to donate to the project so issues we are interested in are fixed?

I don't really want to fork the project and build a new npm under a different name or is it an only option?

Thanks!

olegfeferman avatar Jul 29 '20 11:07 olegfeferman

It's fair to say the project is abandoned.

For the last couple of years, some contributors tried to save the repo but ended up leaving.

The owner came back for a brief period of time. Unfortunately, couple of releases broke more things that fixing bugs. I think most of us, have already fork a stable enough version and move forward on our own. Some have published their forks as new NPM, haven't seen one to become the new go-to.

IceDev-xyz avatar Jul 31 '20 03:07 IceDev-xyz

@IceDev-xyz , thanks for taking time to explain everything!

When you mention "stable enough version" - which version do you mean?

olegfeferman avatar Jul 31 '20 06:07 olegfeferman

I forked v0.65. Was stable for what I needed (overall fitness tracking) and kept working on my own.

IceDev-xyz avatar Jul 31 '20 13:07 IceDev-xyz

This one: https://github.com/IceDev-xyz/rn-apple-healthkit ? - looks empty. In any case thanks, I will look into 0.65

olegfeferman avatar Jul 31 '20 13:07 olegfeferman

... yeah, I ended up working on a private version. A lot of cut-corners I'm not proud of.

IceDev-xyz avatar Jul 31 '20 13:07 IceDev-xyz

Got it. I feel we will go same way... 😄

olegfeferman avatar Jul 31 '20 13:07 olegfeferman

I'll look into taking up the baton, give me a few days as I really need a package like this for long-term

stevelizcano avatar Aug 11 '20 15:08 stevelizcano

I decided to integrate HK directly into my app, as I ran into performance issues with large amounts of data going over the bridge, and I needed background sync functionality.

JulianKingman avatar Aug 11 '20 20:08 JulianKingman

@JulianKingman, sorry for a stupid question but how is it possible to integrate it "directly" into React Native app. Or did you switch to another technology?

olegfeferman avatar Aug 12 '20 05:08 olegfeferman

@olegfeferman I think @JulianKingman means that he is interfacing with Healthkit directly via RN NativeModules.

alfonsosn avatar Aug 26 '20 18:08 alfonsosn

@stevelizcano Have you thought about it? I think first we would need a new main fork. Maybe even get the original owner to link to an "official" continuation of the project.

schumannd avatar Sep 25 '20 08:09 schumannd

@schumannd @stevelizcano I would love to take the lead into this direction too, as I'm going to rely on this library for the long-term. I tried getting in touch with the project owner but I didn't get an answer too

I created a new NPM package to facilitate the development process and also a new repository.

I'm planning in the next few weeks to clean up a few of the documentation (and add proper credits to the original authors) and review a few of the open PRs in this repository, so we can have a central package to work on. Would love to have your help, in case you guys are interested :)

lucaspbordignon avatar Sep 25 '20 18:09 lucaspbordignon

Nice to hear that since I am also planning to work with this library on a long term project. Also good decision to review PRs and merge them, I have seen some nice improvements there.. 👍

devranerogul avatar Sep 27 '20 12:09 devranerogul

@devranerogul That's great to hear! Feel free to give any advice there too and contribute if you want, would definitively love to hear your inputs for the project

lucaspbordignon avatar Sep 27 '20 16:09 lucaspbordignon

Hey everyone, I'm looking to give this project to someone else. I'm no longer consistently working on React Native. If you're interested send me an email at [email protected].

@IceDev-xyz @lucaspbordignon @devranerogul

terrillo avatar Nov 06 '20 14:11 terrillo

Oh, I just saw this after days I opened a ticket and waiting for some helps: https://github.com/terrillo/rn-apple-healthkit/issues/197

I really need to have health kit in my project, really really need to get data sync with my garmin watch.

I'll try v0.65, see if I have any luck getting any demo working.

kenjichanhkg avatar Nov 13 '20 02:11 kenjichanhkg

I'm willing to take the project, I use my own version of this repo (forked from 0.6.5 and kept working on it).

As I use it for my own apps, the focus is on fitness (exercise, steps, activity) more than nutrition, mindfulness and others. But I'll be happy if others contribute on those areas.

I'm thinking of creating a new ReadMe and DemoApp with the functions I focus on, leaving others (ej. getLatestLeanBodyMass) as legacy code.

@terrillo you've got mail.

IceDev-xyz avatar Nov 24 '20 16:11 IceDev-xyz

Hey @IceDev-xyz!

That's great to hear. I've also created a new fork and started to add a few of the functions that you've mentioned (as the improved readme and have a ticket for the demo app)

It's hosted here, would be very happy in case you want to join the repository as one of the core contributors for it

I'm not sure which changes and additions have you made at yours, would love to hear from you so we can join forces on this one!

lucaspbordignon avatar Nov 24 '20 19:11 lucaspbordignon

@lucaspbordignon

It would make sense to only have 1 repo, and yours looks active, I'll compare both forks and will send some PR in order to match repos.

I'll keep mine alive just in case, but will not publish it on NPM.

What version of this repo did you fork?

IceDev-xyz avatar Nov 25 '20 15:11 IceDev-xyz

@IceDev-xyz

For the fork, I've actually made a fork from another one, from nutrisense, as they had a few methods that weren't set in the original repo

I've also added a few of the most recent changes made in this repo on top of that

lucaspbordignon avatar Nov 26 '20 11:11 lucaspbordignon

@lucaspbordignon @IceDev-xyz Just commenting to say thank you for picking this up! Looking forward to using it for mindful minutes 🙏

TowhidKashem avatar Nov 26 '20 18:11 TowhidKashem

@lucaspbordignon @IceDev-xyz I'd like to ask a question here. Which forked version allow me to sync/get workout route data, like GPS tracks and heart-rate data? Because I've been looking around and don't find what I need, or maybe I've missed something.

kenjichanhkg avatar Nov 28 '20 12:11 kenjichanhkg

My fork doesn't support GPS tracks or Heart-rate data.

And I'm pretty sure you can't get workout data with heart-rate, at least this was true couple of years ago when I tried it, the solution is to get the workout data, and then query heart-rate for the start + end timeframe of the workout.

Also, I could be wrong, but pretty sure works the same way for GPS.

https://developer.apple.com/documentation/healthkit/hkworkout/1615212-workoutwithactivitytype?language=objc https://developer.apple.com/documentation/healthkit/hkworkout/workout_metadata_keys?language=objc

IceDev-xyz avatar Nov 28 '20 16:11 IceDev-xyz

@IceDev-xyz I'm trying to write some code to get all the workouts + gps & heart-rate data. But my code doesn't return all the workouts that I did, and I'm struggling about what's the difference of HKSeriesType.workoutRoute() and HKSeriesType.workoutType()

image

kenjichanhkg avatar Nov 29 '20 12:11 kenjichanhkg

Hey if this abandoned and the project has moved on to a repo, could we update this repo's README with the repo we should be focusing on?

jahead avatar Apr 06 '21 04:04 jahead