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 to handle missing multiTenancyConfig key #886

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

edugonza
Copy link
Contributor

@edugonza edugonza commented Feb 19, 2024

fixes #884

multiTenancyConfig key
@dirkkul
Copy link
Collaborator

dirkkul commented Feb 19, 2024

HI @edugonza ,

Thanks a lot for your contribution! Would you be able to add a test for this?

You'd also need to agree to our CLA

@edugonza
Copy link
Contributor Author

edugonza commented Feb 19, 2024

Hi @dirkkul,

I just added a mock test. I agree with the CLA.

@weaviate-git-bot
Copy link

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Slack channel?

@dirkkul dirkkul changed the title fixes weaviate/weaviate-python-client#884 Fix to handle missing multiTenancyConfig key Fix to handle missing multiTenancyConfig key Feb 20, 2024
@dirkkul dirkkul merged commit a33cc27 into weaviate:main Feb 20, 2024
35 checks passed
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.

KeyError: 'multiTenancyConfig'
3 participants