search

Incapamentum / Exalted-Sage

Discord bot that does API requests to retrieve a collection of daily achievements for the next day and pushes an alert when any that is in a watchlist shows up.
3 stars 0 forks source link

Integrate Logger #11

Closed Incapamentum closed 1 year ago

Incapamentum commented 1 year ago

Since the last major update (PR #10), there seems to be some issues with the MessageEventHandler on line 84, despite it being a closing bracket.

Will look into including a logger that will keep track of whether a message is empty or not, and where exactly in execution the code is in that may be causing the failure.

Incapamentum commented 1 year ago

Looked into existing logging solutions. One that seemed the simplest without any kind of negative connotation attached to it was NLog. NuGet package was available, and installed that way.

The main idea in including this logger is for every event handler to maintain their own private, static copy of a logger. Config file applies to all loggers. Will go ahead and create a PR soon.