[x] If this PR is a new feature, please reference a discussion where a consensus about the design was reached (not necessary for small changes)
[x] Make sure all the significant new logic is covered by tests
[x] Try to include the Pull Request inside of CHANGELOG.md
I was following the documentation about multiple clients, I encountered an error stating Client has not been defined yet.
The documentation states that if you have a named option default, it will be the default client. That does not work.
Currently, if you provide options for a default client through APOLLO_NAMED_OPTIONS, it creates a "named client" named default. Accessing it is not possible because the actual default client is not set, and using use('default') results in the default client.
This fixes that. The test added fails without the fix.
Checklist:
I was following the documentation about multiple clients, I encountered an error stating
Client has not been defined yet
.The documentation states that if you have a named option
default
, it will be the default client. That does not work.Currently, if you provide options for a
default
client throughAPOLLO_NAMED_OPTIONS
, it creates a "named client" nameddefault
. Accessing it is not possible because the actual default client is not set, and usinguse('default')
results in the default client.This fixes that. The test added fails without the fix.