Why does the Pedometer only expose step count? Can we update to return all CMPedometerData fields?

[kevinwalsh23]

Please provide the following:

1. SDK Version: 35
2. Platforms(Android/iOS/web/all): iOS

CMPedometerData exposes a few other fields that would be incredibly valuable to a lot of devs, including distance and currentPace. Why did we choose to limit this to just number of steps?

[adamjnav]

Hey @kevinwalsh23,

When we authored the module we likely went for core functionality to be able to get it out in a timely manner for release. I would recommend creating a feature request to expose these additional fields on our Canny Board.

Cheers,
Adam

[kevinwalsh23]

Hey @adamjnav,

Thanks for the response. Created the ticket here: Update Pedometer to return all CMPedometerData fields | Voters | Expo.

Updating should be as simple as changing this file, ‘expo/ABI35_0_0EXPedometer.m at 1844c78d3815558378f201c78554598968e24773 · expo/expo · GitHub’, and changing ‘body:@{@“steps”: pedometerData.numberOfSteps}]’ to just '‘body:@{@“pedometerData”: pedometerData}]’ and ‘resolve(@{@“steps”: pedometerData.numberOfSteps});’ to ‘resolve(@{@“pedometerData”: pedometerData});’.

Thanks,
Kevin

[hugo_ladeira]

@adamjnav sorry for doing this, but…
Can you take a look in my case?

Location.startLocationUpdatesAsync don't start in real device with apk

FWIW I tried to get this working yesterday and failed. I wondered if maybe Toasts don’t work from an app that’s in the background, so I tried using network requests. Now that I think about it, BackgroundFetch probably exists for a reason so maybe just using fetch() was not a valid way to test this. Are you sure that ToastAndroid works from a backgrounded app?

[kevinwalsh23]

@adamjnav If I submitted a pull request for the feature change, would you be able to review this week and merge?

Could go about it in three ways, let me know what Expo prefers.

1. Alter the current Pedometer module, exposing all of the fields. (May alter applications currently using the Pedometer as is)

2. Alter the current Pedometer to return number of steps in the same manner (applications currently using this module will be unaffected), but also include the other fields in a similar fashion.

3. Create a new Module that returns all of the fields, and leaves the current Pedometer unchanged.

Let me know! Very close to finishing an application that requires this, so glad to just get this work done.

Kevin