Hi @beeman, just checking in with you, could you provide me a status on this PR?

@Jefiozie it got stuck due to a discussion we had what public meant.
There are two options:

1. Public has no meaning and should be removed.
2. Public should be used for explicit public api while non public is used for angular implicit public methods like ngOnInit.

Well, this is about code esthetics, as there is no actual difference in having the public in place.
I would simply remove all those keywords as its just visual noise.

Before merging this we should resolve the conflict.

Also I would opt to add a eslint rule that explicit defines what should be defined and what not. In this case I would also opt to remove all public keywords.

I will add a comment on the linting issue to make sure that we add this rule.

@Jefiozie Done, also removed a couple more stray 'puclic''s ;)