Handy-Hangout-Studios / Norm

GNU General Public License v3.0
2 stars 1 forks source link

Convert DI Configuration and Creation to use Vertical Slices #17

Open ProfDoof opened 3 years ago

ProfDoof commented 3 years ago

One problem noted by TeBeCo in the C# server is that I add a bunch of services that are theoretically each supposed to be an individual feature. Thing is that they aren't they are really a collection of features that are just supporting BotService which means they should be in the bot services context instead. 🤔 This needs a rework. Right now, BotService, ModerationService, EventService, etc. are circularly dependent on each other and that needs to end.

ProfDoof commented 3 years ago

He also noted that my naming could be better and instead of Configure, it should be called Add since I'm not doing many configurations in there.