sync scheduler overhaul

[carykees98]

No description provided.

[lavajuno]

Good stuff, looking forward to getting this up and running.

[Warvan1]

Its hard to tell what parts of this is actually changing functionality because 99% of the diff is just changing the code formatting. (the new formatting is also harder to read than the old formatting)

[lavajuno]

I think a lot of this is refactoring to make it easier to add/change functionality later. We still need to do some work on manual sync before it's up and running, we're also experiencing a lot of #13 in production.

[Warvan1]

why did you change all the function definitions to the following format?

auto Schedule::getInstance() -> Schedule*
{
//code
}

this is WAY harder to read especially for newer students. Also I like to avoid auto (except for some cases like range for loops).

Also I am resisting the urge to nitpick all of the other weird formatting but I couldn't ignore this.