-
Notifications
You must be signed in to change notification settings - Fork 24
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
Added NLog 4.6 is live post #99
Conversation
Thanks in advance! |
Guess some example code should be added to the |
Added details about use of norawvalue + parameterType
Using XmlLayout with MSSQL XML Column
Simpler Layout for appsettings.json example
@snakefoot I've added also some sections . Feel free to change the grammar when you spot some mistakes. |
``` | ||
|
||
When using a non-string DbType then it will attempt to convert the raw value directly to the expected type. | ||
When using special DbType like `SqlDbType.SmallInt` then one can specify `parameterType="Int16"` to help. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is indeed important, but maybe it should be only in the wiki and not in the release post. (it is very much in-depth). Is that OK with you?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. Think I was a little too happy about the change :)
Removed nitty gritty details about norawvalue
Include size="-1" for SqlDbType.Xml
dependencies on System.Messaging (MSMQ). Since NLog already dependends on System.Configuration | ||
then it didn't make sense also to need NLog.Extended for reading app.config / web.config. | ||
|
||
### NLog configuration inside appsettings.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to merge NLog/NLog.Extensions.Logging#263 first for this?
(and mention NLog.Extensions.Logging 1.5?) -
Doubting if this should be a separate post.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You decide. There is no law saying that the news-posts should be posted on the same day as the release is cut.
Initial draft:
https://github.com/snakefoot/NLog.github.io/blob/patch-3/_posts/2019-02-30-nlog-4-6-is-live.md