Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document simple logging (DbContextOptionsBuilder.LogTo) #2085

Closed
ajcvickers opened this issue Jan 31, 2020 · 2 comments · Fixed by #2740
Closed

Document simple logging (DbContextOptionsBuilder.LogTo) #2085

ajcvickers opened this issue Jan 31, 2020 · 2 comments · Fixed by #2740

Comments

@ajcvickers
Copy link
Member

See dotnet/efcore#15403 (comment)

@jwooley
Copy link

jwooley commented Oct 5, 2020

By setting this as "Undocumented feature" does that mean that LogTo is not planned to be a supported solution going forward?

@ajcvickers
Copy link
Member Author

@jwooley No. It means it needs to be documented.

ajcvickers added a commit that referenced this issue Oct 6, 2020
Fixes #2085

Note that this is the first part of several new pages/updates for logging and interception. For now, I'm linking to the new page from the old top-level logging. As more of this gets done that will gradually be replaced. I didn't add anything to the table of contents yet so as not to make this too confusing.
ajcvickers added a commit that referenced this issue Oct 8, 2020
Fixes #2085

Note that this is the first part of several new pages/updates for logging and interception. For now, I'm linking to the new page from the old top-level logging. As more of this gets done that will gradually be replaced. I didn't add anything to the table of contents yet so as not to make this too confusing.
ajcvickers added a commit that referenced this issue Oct 9, 2020
Fixes #2085

Note that this is the first part of several new pages/updates for logging and interception. For now, I'm linking to the new page from the old top-level logging. As more of this gets done that will gradually be replaced. I didn't add anything to the table of contents yet so as not to make this too confusing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants