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

Basic workout events #177

Merged
merged 3 commits into from
Aug 4, 2019
Merged

Basic workout events #177

merged 3 commits into from
Aug 4, 2019

Conversation

tsoporan
Copy link
Owner

@tsoporan tsoporan commented Aug 4, 2019

Related #159

Proposed Changes

  • A WorkoutEvent model for tracking state changes on workouts
  • Accompanying helpers with some sprinkled type hints
  • Not entirely happy the templating/state structure for actions but decent first pass

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
@tsoporan tsoporan self-assigned this Aug 4, 2019
Signed-off-by: Titus Soporan <titus@tsoporan.com>
@tsoporan tsoporan merged commit 8871f8d into master Aug 4, 2019
@tsoporan tsoporan deleted the 159-workout-events branch August 4, 2019 18:51
@tsoporan tsoporan added this to the v1.4 milestone Aug 4, 2019
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.

1 participant