se701g2 / Doto

Todo application for Group 2
https://doto.azurewebsites.net/
MIT License
12 stars 42 forks source link

issue/114-setup a formal logging system closes #114 #207

Closed jordansimsmith closed 4 years ago

jordansimsmith commented 4 years ago

Used the popular winston logging library to configure logging for doto-backend. To use, import the logger from the logging package const { logger } = require('/path/to/logging'); and use logger.warn / info/ error as appropriate. Logging has also been configured as middleware to work with express to log the incoming requests.

closes #114

Matteas-Eden commented 4 years ago

Not relevant to the logger, but: The description uses github syntax to link to the issue. E,g Resolves se701g2/Doto#{Number} is actually unfulfilled by your description. While there is a mention of the issue in the title of the PR, GitHub explicitly requires that such links be present in the description to work.