Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Replace All console.log Messages #2052

Open
justinr86 opened this issue Jun 19, 2020 · 4 comments
Open

Replace All console.log Messages #2052

justinr86 opened this issue Jun 19, 2020 · 4 comments
Assignees
Labels
medium priority not high, not low, but medium!
Projects

Comments

@justinr86
Copy link
Contributor

Summary

Now that the update to Winston Logger has been merged we can add the logger to the rest of the application. All console.log messages should be replaced in server-side code.

Design detail

Replace console.log with logger.info/warn/error as applicable. This change should be made to server side code.

Unresolved questions

N/A

@justinr86 justinr86 added the needs prioritization We have to prioritize this issue label Jun 19, 2020
@justinr86
Copy link
Contributor Author

The update to Winston Logger needs to be tested against Azure analytics. Work cannot begin until proper testing is complete.

@stephaniemgauthier stephaniemgauthier added medium priority not high, not low, but medium! and removed needs prioritization We have to prioritize this issue labels Jun 22, 2020
@lmcbhvu lmcbhvu added this to Backlog in Sprint 1 Jun 24, 2020
@stephaniemgauthier
Copy link

Who do we need to do the testing?

@justinr86
Copy link
Contributor Author

@davidheerema raised this issue, when he is back I will ask him about it.

@justinr86 justinr86 removed the blocked label Jul 16, 2020
@justinr86
Copy link
Contributor Author

Since the latest update to logger was merged this issue is no longer blocked.

@dengcn dengcn self-assigned this Aug 11, 2020
@dengcn dengcn removed their assignment Oct 26, 2020
@justinr86 justinr86 self-assigned this Oct 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
medium priority not high, not low, but medium!
Projects
Sprint 1
  
Backlog
Development

No branches or pull requests

3 participants