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

Separate settings for API, Callback-Handler and Image-Handler #29

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

loechel
Copy link
Contributor

@loechel loechel commented Feb 7, 2025

When we use the Handlers standalone it makes not sense to supply a credential file to the application. But that is required by the settings for API and Session.

@loechel loechel requested a review from jensens February 7, 2025 13:59
@loechel loechel self-assigned this Feb 7, 2025
@jensens
Copy link
Contributor

jensens commented Feb 8, 2025

Too many refactoring in here, this would bump a major version. The same can be achieved with more minimal impact.

First it would be good to have a test showing the problem. Then we can think of a minimal impact solution.

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