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

Fix provider fallbacks #897

Merged
merged 2 commits into from Aug 6, 2021
Merged

Fix provider fallbacks #897

merged 2 commits into from Aug 6, 2021

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented Aug 6, 2021

In #830 I broke the fallback list after fixing a linter error wittout looking at any of the context of the line I was fixing... (and then rushed through the PR as people were on holiday, my bad)

This PR makes the providers list importable from the server.routers.utils module too.

Closes #896.

@ml-evs ml-evs added bug Something isn't working priority/high Issue or PR with a consensus of high priority server Issues pertaining to the example server implementation labels Aug 6, 2021
@ml-evs ml-evs enabled auto-merge (squash) August 6, 2021 10:44
@ml-evs ml-evs merged commit fd78ce2 into master Aug 6, 2021
@ml-evs ml-evs deleted the ml-evs/fix_#896 branch August 6, 2021 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority/high Issue or PR with a consensus of high priority server Issues pertaining to the example server implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provider fallbacks are still not working
2 participants