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

feat: add new values for session & user profile #40

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

mbpzn
Copy link

@mbpzn mbpzn commented Dec 19, 2024

Based on fitsdk/Profile

Extracting two new fields from user_profile:

def: 28
name: wake_time
type: localtime_into_day
comment: Typical wake time

def: 29
name: sleep_time
type: localtime_into_day
comment: Typical bed time

And two new fields from session:

def: 192
name: workout_feel
type: uint8
comment: A 0-100 scale representing how a user felt while performing a workout. Low values are considered feeling bad, while high values are good.

def: 193
name: workout_rpe
type: uint8
comment: Common Borg CR10 / 0-10 RPE scale, multiplied 10x.. Aggregate score for all workouts in a single session.

@krzysztof-cislo
Copy link

Hi @jimmykane !

It's us again. 🙂 Can you merge the PR and release new version, please?

Thanks!

@jimmykane
Copy link
Owner

Will do asap. Put a reminder for tomorrow as now I am on the road again is that ok?

@krzysztof-cislo
Copy link

Will do asap. Put a reminder for tomorrow as now I am on the road again is that ok?

Hi @jimmykane I know that it's holiday time, but if you can find some time to create a release we will be very grateful.

@krzysztof-cislo
Copy link

Will do asap. Put a reminder for tomorrow as now I am on the road again is that ok?

Hi @jimmykane I know that it's holiday time, but if you can find some time to create a release we will be very grateful.

Hi again @jimmykane ! Any chance to merge this ASAP? 🙏

@jimmykane jimmykane merged commit f76c86a into jimmykane:master Jan 2, 2025
@jimmykane
Copy link
Owner

jimmykane commented Jan 2, 2025 via email

@krzysztof-cislo
Copy link

Thank you! 🙂

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.

3 participants