-
Notifications
You must be signed in to change notification settings - Fork 2
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
#187419126 A seller should be able to update & delete a product item from their collection #14
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
}`} | ||
> | ||
{checked ? ( | ||
<svg |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
/> | ||
</svg> | ||
) : ( | ||
<svg |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #14 +/- ##
==========================================
- Coverage 78.86% 75.92% -2.95%
==========================================
Files 56 57 +1
Lines 899 976 +77
Branches 96 107 +11
==========================================
+ Hits 709 741 +32
- Misses 186 233 +47
+ Partials 4 2 -2 ☔ View full report in Codecov by Sentry. |
013cb19
to
51b4387
Compare
51b4387
to
b3e3e3c
Compare
b3e3e3c
to
64dc819
Compare
64dc819
to
7cb41c0
Compare
7cb41c0
to
24bcf2c
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
24bcf2c
to
21d101c
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
- a seller should be able to update a product from their collection - a seller should be able to delete a product from their collection Delivers #187419126
21d101c
to
73d3859
Compare
Code Climate has analyzed commit 73d3859 and detected 0 issues on this pull request. View more on Code Climate. |
What does this PR do?
This PR Adds product update and deletion of products by sellers
Description of Task to be completed?
How should this be manually tested?
ft-product-update-delete
.npm install
to install dependencies.npm run dev
to start the development server and login as a seller/dashboard/products
What are the relevant pivotal tracker stories?
Delivers #187419126
Screenshots
Design
Implementation