I will write here all the improvements based on PR #44 - [x] [H: too late should be in base class before any verification of parameters. Same for similar commands. Code duplicated as well. ](https://github.com/Moe98/reddit/pull/44#discussion_r628933336) - [x] [M: function belongs to Arango module. It should be there.](https://github.com/Moe98/reddit/pull/44#discussion_r628798370) - [ ] [M: could add a method prestart to Service and override it in this class to ensure uniformity across apps.](https://github.com/Moe98/reddit/pull/44#discussion_r628931331) - [ ] Add uriParams validation in CommandWithVerification - [ ] Builder Pattern for User model - [ ] PostgresConnection is achimera of static and instance methods. Not fully a static class and not fully a Singleton class either. - [ ] Proper testing of PostgresConnection - [ ] Main Method should not throw an exception - [ ] graceful shutown if env variable not loaded properly - [ ] ~~Use Attribute class to get the parameter from the body~~ - [ ] use try-with-resources or close this AutoCloseable ](https://github.com/Moe98/reddit/pull/44#discussion_r628943399) - [ ] Transaction-like behavior when using 2 different DBs - [ ] [Users' deletion discussion](https://github.com/Moe98/reddit/pull/61#issuecomment-840497186)
I will write here all the improvements based on PR #44
Use Attribute class to get the parameter from the body