-
-
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
feat: customer module skeleton #6126
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
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 work 🔥
heuuu, I am very sorry @srindom , I thought it was ready to review when i received the email and seen it open 😂 |
@adrien2p - please take another look when you have time |
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 🔥 couple of todo's that ca also be done later
}) | ||
updated_at: Date | ||
|
||
@Property({ columnType: "timestamptz", nullable: true }) |
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.
todo: missing index
metadata?: Record<string, unknown> | null | ||
} | ||
|
||
export type UpdateAddressDTO = { |
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.
todo: missing id I think
@@ -0,0 +1,28 @@ | |||
export type CreateAddressDTO = { |
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.
todo: change type to interfaces as per our discussion
metadata?: Record<string, unknown> | null | ||
} | ||
|
||
export type UpdateCustomerGroupDTO = { |
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.
todo: move from type to interface, then we could have something like
export interface UpdateCustomerGroupDTO extends Partial<CreateCustomerGroupDTO> {
id: string
}
metadata?: Record<string, unknown> | ||
} | ||
|
||
export interface UpdateCustomerDTO { |
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.
export interface UpdateCustomerDTO { | |
export interface UpdateCustomerDTO extends CreateCustomerDTO { | |
id: string | |
} |
No description provided.