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(planner): add ability to check-off steps #296

Closed
wants to merge 1 commit into from
Closed

feat(planner): add ability to check-off steps #296

wants to merge 1 commit into from

Conversation

coogie
Copy link

@coogie coogie commented Oct 24, 2020

Hihi,

Started using SA again recently while waiting for SL to drop, so decided to contribute back to it. Been a long time since I touched AngularJS, so bear with me 😄

New

  • Adding functionality to check-off "runs" in the mount planner.
  • Create mountRuns directive to handle common presentation and logic of planner tables
  • Create plannerStepHasRun to perform filtering on planner step based on new hasRun bool

Changes

  • Refactor MountsCtrl to remove large amount of code migrated to mountRuns directive
  • Add hasRun bool to planner steps for handling user-determined completion of step
  • Add visualIndex to planner steps to preserve "Step #" as it moves between complete/incomplete state

In the future, I'd hope to expand this with localStorage to preserve progress through sessions and page reloads, but I wouldn't like this PR getting much larger than it is

image

Closes #249
Closes #267

@coogie
Copy link
Author

coogie commented Oct 27, 2020

Impatiently pinging @seirl because it's almost Wednesday and time for another run 👀 😅

@Goten87
Copy link

Goten87 commented Feb 10, 2021

now wait untill someone comes along to add it please someone add this this to the website already >_<

@coogie
Copy link
Author

coogie commented Feb 10, 2021

Yeah, I got bored of waiting for this PR to land, so I build my own planner with this feature 😅
https://planner.overlazy.com

@coogie coogie closed this Feb 10, 2021
@coogie coogie deleted the feat/planner-checklist branch December 14, 2021 00:47
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.

way to track what bosses you did To Do List Functionality
2 participants