Ariel Weinberger

@yuval-hazaz Please prioritize

@abrl91 What's the status on this? Can we close it?

Tested on Next and works properly.

@yuval-hazaz Please prioritize

Hi, sorry for the late response and thanks for your PR. The expectation is that the Amplication NestJS logger will be the exclusive logger module used in the server.

I see, @ParkourKarthik, thanks for pointing out. The goal for this issue is that all other forms of logging (default NestJS `Logger`, `winston` etc.) will be gone, and our package...

@ParkourKarthik You would need to remove any usage of `winston` there and perform dependency injection to the Amplication NestJS Logger, and replace the `winston` calls there with those of our...

@tupe12334 I see you approved this change. Have you tested it and made sure it works properly?

I've added some more comments. The intention is to not use Winston in any of our services, and importing Winston types into our service binds us to the Winston API....

I'm happy to accept it as-is, it will still be of great value for us! Thank you @ParkourKarthik