-
Notifications
You must be signed in to change notification settings - Fork 125
Incorrect AppSettings key #38
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
Comments
Thanks for the note! I've fixed the use of |
Hi. The RollingInterval.Day was my mistake: I thought -don't know why- that enumerations have to be fully specified (RollingInterval.Day instead of just Day). Could you check the most recent open issue in the Email sink? Not reported by me but there is a problem there when the user is authenticated but the server expects anonymous access. Thanks again. |
👍 thanks for the follow-up. The email sink is in need of some extra help, I'd love to dig into it but realistically probably won't get the time to figure this one out. All help appreciated over there! :-) Cheers |
@nblumhardt , I hate to comment on an already closed issue....but I think it may be a good idea to specify the gotcha about the enum value in the README because that is not well documented elsewhere I've looked. At least, add in the code snippet that also configures the |
rollingInterval : 1 works |
When using the AppSettings package the correct key for the file path is "path" not "pathFormat" as the documentation says (leftover from previous version). Furthermore, I am getting a RollingInterval.Day not found error, but "Day" (or 3) seems to be working fine.
Best regards, and thanks!
The text was updated successfully, but these errors were encountered: