Onboarding Project

[JeanNicholson]

No description provided.

[FritzOnFire]

Now, there is nothing wrong with having more than one class in the same file, but you have a lot of other logic mixed in with your classes, and app.ts is clearly where you app starts and not where you should be defining classes. Please try and create some more files

[jasonkofo]

I got a bit tired, especially because I am sure @FritzOnFire is going to cover the other style issues in his PR review. I chose to simply overlook those issues, because it doesn't seem that this code was developed in accordance with the style guide - which is okay until the next sweep of the PR.

A lot of the comments I made are suggestions about the general code design. Please feel free to ask me what I meant if some of the comments that I might give off an air of insanity, or pedantry, because that unfortunately is a thing that often happens.

[FritzOnFire]

Some formatting comments, but you are done in my opinion :D

[FritzOnFire]

Please put this on the same line.

[FritzOnFire]

please put a ; at the end of this line.

[FritzOnFire]

this can be html += "<tr>";

[FritzOnFire]

this can be html += "<td>" + cell + "</td>";

[FritzOnFire]

this can be html += "</tr>";

[FritzOnFire]

this can be html += "</tr>";