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

New workout flow #167

Merged
merged 27 commits into from
Jan 21, 2019
Merged

New workout flow #167

merged 27 commits into from
Jan 21, 2019

Conversation

tsoporan
Copy link
Owner

@tsoporan tsoporan commented Jan 19, 2019

Fixes #162

Proposed Changes

  • Added and repaired tests and CI
  • Factored out add custom exercise dialog
  • NewWorkout dialog which allows staging a workout
  • Restructured graphql back-end to avoid circular import issues

Signed-off-by: Titus Soporan <titus@tsoporan.com>
@tsoporan tsoporan added this to the v1.4 milestone Jan 19, 2019
@tsoporan tsoporan self-assigned this Jan 19, 2019
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Updates to all previous failing tests

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
- Test with coverage
- Bump circleci version
- Clarify workflow

Signed-off-by: Titus Soporan <titus@tsoporan.com>
- Down to 2.0, advanced feature needs to be enabled for 2.1
- Updates to jobs

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Significant changes to structure, workflow and styling. Currently using
mock data.

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
A pending workout is immediately created when you hit '+'

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
The previous structure lended itself to circular import issues. This
also looks closer to the front-end.

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
- Also remove uuid as we can now rely on exercise type id

Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
Signed-off-by: Titus Soporan <titus@tsoporan.com>
@tsoporan tsoporan merged commit 91b7b3a into master Jan 21, 2019
@tsoporan tsoporan deleted the 162-new-workout-flow branch January 21, 2019 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant