-
-
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
breaking: remove deprecated commands and code #9521
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
Should we include the following
|
@adrien2p Include where? |
oh no worries I think you did it already to remove the deprecated inventory and stock |
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.
Very, very nice
@@ -397,6 +336,12 @@ function buildLocalCommands(cli, isLocalProject) { | |||
describe: process.env.PORT | |||
? `Set port. Defaults to ${process.env.PORT} (set by env.PORT) (otherwise defaults ${defaultPort})` | |||
: `Set port. Defaults to ${defaultPort}`, | |||
}) | |||
.option(`cluster`, { |
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.
nice!
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.
not related to this PR, but the singular
plural
properties at some point will have to be added somewhere, as we auto generate methods based on them, non english names won't work, and non standard english plurals won't work properly.
Fixes: FRMW-2736
This PR removes deprecated commands and code from the Medusa monorepo. Here's a quick summary of it. Please, feel free to review this PR thoroughly to ensure I am not breaking something by mistake :)
Commands removal
seed
: There was no implementation for theseed
command in the first place.migrations [action]
: Removed in favor ofdb:migrate
,db:rollback
anddb:generate
.links [action]
: Removed in favor ofdb:sync-links
.start-cluster
: Removedstart-cluster
command in favor of the--cluster
flag on the start command.Config options removal
databaseType
as it is not used anywhere.httpCompression
in favor ofcompression
property.allowedFields
in favor ofallowed
property from theMedusaRequest
.expand
property in favor ofallowed
property from theRequestQueryFields
.defaultFields
anddefaultRelations
in favor ofdefaults
property from theRequestQueryFields
.allowedFields
andallowedRelations
in favor ofallowed
property from theRequestQueryFields
.Events build changes
eventsEnum
in favor ofeventName
property accepted by themoduleEventBuilderFactory
. Also updated all the usages to useeventName
.ModelDTO changes
singular
andplural
fromModelDTOConfig
.Models template changes
create
,update
,singular
, andplural
fromModelsConfigTemplate
.Zod validator changes
OptionalBooleanValidator
in favor ofbooleanString
.