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

LoadConfigRoot doesn't handle config entries with null values #199

Closed
AnashOommen opened this issue Dec 18, 2018 · 1 comment
Closed

LoadConfigRoot doesn't handle config entries with null values #199

AnashOommen opened this issue Dec 18, 2018 · 1 comment
Assignees
Labels
bug Something isn't working P1 usability This issue is related to a usability issue with the client library.

Comments

@AnashOommen
Copy link
Member

AnashOommen commented Dec 18, 2018

See googleads/google-ads-dotnet#26 for broader context. #191 also alludes to this issue.

Basically, the AdWordsAppConfig class should handle gracefully the situations where a config key has a null value in the config file.

@AnashOommen AnashOommen self-assigned this Dec 18, 2018
@AnashOommen AnashOommen added bug Something isn't working usability This issue is related to a usability issue with the client library. P1 labels Dec 18, 2018
@AnashOommen
Copy link
Member Author

Fixed in 24.3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1 usability This issue is related to a usability issue with the client library.
Projects
None yet
Development

No branches or pull requests

1 participant