-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
chore: Update modules providers configuration with 'identifier' and 'PROVIDER' #9636
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Skipped Deployments
|
|
cc @sradevski @christiananese you might need to take a look at this one, we can discuss if you need any help for the changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. This shouldn't affect us I believe, we don't have any custom providers implemented for now.
It affect the core providers as well, the auth providers have been aligned as well, maybe that can impact you? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wonder if we should snake_case the IDs for consistency? It will be a breaking change regardless, so thought we might as well
I don't think this breaks the auth domain, because the |
-- In order to normalize and be consistent, now the auth also concatenate the identifier and id which was not the case before, that can be breaking too -- nvm the above, we didn't, but I thought we did 😂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…PROVIDER' (medusajs#9636) * chore: Update modules providers configuration with 'identifier' and 'PROVIDER' * update check * fix tests * type * normalize auth provider * emailpass * providers --------- Co-authored-by: Carlos R. L. Rodrigues <[email protected]> Co-authored-by: Carlos R. L. Rodrigues <[email protected]>
What
identifier
static property. Now all providers must have a static propertyidentifier
which will be used to register the provider into the module container.id
in the providers configuration can still be provided in the defineConfig if some provider can have multiple instances (e.g stripe per region)pp_stripe_the-given-id
orpp_stripe
instead ofpp_stripe_
orpp_stripe_undefined
Breaking changes
region_provider_id
might be broken after the provider are reloaded since their id have been fixed. You might need to update those manually or regenerate those link