Skip to content

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

Closed
rmbrunet opened this issue Oct 30, 2017 · 5 comments
Closed

Incorrect AppSettings key #38

rmbrunet opened this issue Oct 30, 2017 · 5 comments

Comments

@rmbrunet
Copy link

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!

@nblumhardt
Copy link
Member

Thanks for the note! I've fixed the use of pathFormat in the README, but I can't spot "RollingInterval.Day" in there (seems to use "Day") - if I've missed one, could you please point me to it? Many thanks! :-)

@rmbrunet
Copy link
Author

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.

@nblumhardt
Copy link
Member

👍 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

@tafs7
Copy link

tafs7 commented Jan 18, 2018

@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 property from appSettings.

@AlexaCodex
Copy link

rollingInterval : 1 works

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants