Closed ReubenBond closed 1 year ago
This is great work, you guys nailed it. I'll review and test it.
@erjain merge when ready
I think this update means that logs are no longer appearing in Seq. Is this intentional? There still is a Seq container in docker-compose.yml
but its not picking anything up
Yes it’s intentional. We’re also not done with the clean up. This was just the first big set of changes
This is a joint effort between @davidfowl & myself to modernize and clean up eShop. There is more to do, but we wanted to merge what we have so far before we continue.
@erjain, we have tested this locally in VS (launch & manually test + automated tests) and via docker compose, but we haven't tested beyond that (Kubernetes, Azure). I don't expect you to review this because the PR is far too large, but if you could run some of your own tests, that would be greatly appreciated.
Addresses #2070