Add props to components

[atompie]

We have been developing very fast. A lot of components does not have propTypes set. The code needs some factoring and the props would be very useful.

Just pick one component and add props definition.

[danjohnson77]

Happy to take a run at these. Just to confirm you want PropTypes added?

[atompie]

@danjohnson77 Yes I do :)

[danjohnson77]

Gotcha, just wrapping a build but can get into it in the next couple days, thanks!

[atompie]

@danjohnson77 Sure thing - no problem :)

[danjohnson77]

@atompie Hi just checking in, did anything need correcting on the PR?

[atompie]

@danjohnson77 The props are fine, but there are a lot of other changes regarding code formatting. Probably the changes were made by your IDE. I you could just commit the changes in props that could be a lot easier for me to review the PR.

[danjohnson77]

@atompie Understood, yeah looks like prettier got ahold of it. Will redo and submit new PR thanks

[atompie]

@danjohnson77 thanks. :)

[danjohnson77]

@atompie I'll also continue on this issue, do you have a preference on how many components are in a PR? As in, should I do just a few at a time and submit? Or do all of them in one big PR?

[atompie]

@danjohnson77 I do not have any preferences. You may do it as it suits you :)

[atompie]

@danjohnson77 as to new issues, we have some places in UI that need new components. I will describe it and get back to you. I can also give you a hand if you need a help with on boarding to the project.

[danjohnson77]

@atompie Sounds great!