-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feature/Refactor: product forms #309
Conversation
9f9102e
to
785da03
Compare
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.
Please rebase this first, as now reviews are done on things that are already merged to develop.
apps/dashboard/src/components/container/ContainerActionsDialog.vue
Outdated
Show resolved
Hide resolved
6e10205
to
1373e3e
Compare
Rebased and should be good now! |
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.
Some nitpicks, some things to think about, some swagger mismatches.
apps/dashboard/src/modules/seller/components/ProductActionExistingProductForm.vue
Outdated
Show resolved
Hide resolved
apps/dashboard/src/modules/seller/components/ProductActionExistingProductForm.vue
Outdated
Show resolved
Hide resolved
apps/dashboard/src/modules/seller/components/ProductActionExistingProductForm.vue
Outdated
Show resolved
Hide resolved
apps/dashboard/src/modules/seller/components/ProductActionExistingProductForm.vue
Outdated
Show resolved
Hide resolved
1a9c078
to
eeefedd
Compare
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 PR is included in version 1.4.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Warning: this pull request should be merged after #229
Refactor products forms to use the new form way, increases stability and uniform design, while preserving the reactivity that the product dialogs had. Also introduces a delete button for products.
Description
Mostly behavior is the same, slight changes is the ability to delete products. In container context they only remove from the container, without a container context they will delete the product, behind a confirm dialog.
Also fixed two bugs (#300 and #308).
Related issues/external references
#273 (one of many)
Closes #308
Closes #300
#263 (one of few)
Types of changes