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

Feature/Refactor: product forms #309

Merged
merged 14 commits into from
Aug 28, 2024
Merged

Feature/Refactor: product forms #309

merged 14 commits into from
Aug 28, 2024

Conversation

SuperVK
Copy link
Member

@SuperVK SuperVK commented Aug 18, 2024

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

image
image
image

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.

image

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

@SuperVK SuperVK changed the title Refactor: product forms Feature/Refactor: product forms Aug 18, 2024
@SuperVK SuperVK force-pushed the feature/improved-seller branch from 9f9102e to 785da03 Compare August 22, 2024 13:51
Base automatically changed from feature/improved-seller to develop August 22, 2024 13:59
Copy link
Contributor

@CodeNamedRobin CodeNamedRobin left a 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.

@SuperVK
Copy link
Member Author

SuperVK commented Aug 23, 2024

Rebased and should be good now!

Copy link
Contributor

@CodeNamedRobin CodeNamedRobin left a 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.

Copy link
Contributor

@CodeNamedRobin CodeNamedRobin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@SuperVK SuperVK merged commit 78f4449 into develop Aug 28, 2024
3 checks passed
@SuperVK SuperVK deleted the fix/product-form branch August 28, 2024 10:08
Copy link

🎉 This PR is included in version 1.4.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Manage products calls the backend too much [Bug]: Sometimes create new product on container bugs
2 participants