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

Update configuration handling #99

Merged
merged 4 commits into from
Sep 12, 2024
Merged

Update configuration handling #99

merged 4 commits into from
Sep 12, 2024

Conversation

NeonDaniel
Copy link
Member

@NeonDaniel NeonDaniel commented Aug 29, 2024

Description

Update warning log to use log_deprecation
Update config handling to support api_services config outside of keys since #98 enabled more configuration options

Issues

Continues config options added in #98

Other Notes

This leaves backwards-compat for config but allows configuring api_services outside of keys configuration since config may contain more options than keys now. i.e.:

api_services:
  wolfram_alpha:
    api_key: KEY
    cache_timeout: 300

Update config handling to support `api_services` config outside of `keys` since #98 enabled more configuration options
neon_api_proxy/controller.py Outdated Show resolved Hide resolved
@NeonDaniel NeonDaniel force-pushed the FEAT_UpdateConfigHandling branch from 95a41a2 to 3beed12 Compare September 12, 2024 01:45
@NeonDaniel NeonDaniel merged commit 4c09642 into dev Sep 12, 2024
8 checks passed
@NeonDaniel NeonDaniel deleted the FEAT_UpdateConfigHandling branch September 12, 2024 01:52
@github-actions github-actions bot mentioned this pull request Sep 23, 2024
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

Successfully merging this pull request may close these issues.

2 participants