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

[BUG] Multitenancy Config | Tenant Code Resolver #168

Closed
oakbani opened this issue Jan 16, 2023 · 3 comments
Closed

[BUG] Multitenancy Config | Tenant Code Resolver #168

oakbani opened this issue Jan 16, 2023 · 3 comments

Comments

@oakbani
Copy link

oakbani commented Jan 16, 2023

There are multiple issues with the tenant code resolver as given in the multi tenancy config docs in medusa extender.

image

  1. medusa-config is a js file with doesn't accept type annotations ( req: MedusaRequest)
  2. If you pass on a simple method here, medusa-extender keeps throwing this error

Tenant Code Resolver is not a function

Even tried with exactly how you have used in your unit test.
multi_tenancy: { enable: true, tenant_code_resolver: (req) => req.headers['x-tenant'],

@adrien2p
Copy link
Owner

Thanks for the reporting, it should be fixed by #169

@adrien2p
Copy link
Owner

Fixed in the next release. I ll be away fir a week

@oakbani
Copy link
Author

oakbani commented Feb 1, 2023

Thanks man!

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

No branches or pull requests

2 participants