-
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
#187900465: A buyer and seller should be manage orders #24
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
9c7bc3f
to
1f1c2e7
Compare
1f1c2e7
to
33da498
Compare
dispatch(updateOrderStatus({ orderId, status })); | ||
}; | ||
|
||
return ( |
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.
Avoid too many return
statements within this function.
); | ||
} | ||
|
||
return ( |
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.
Avoid too many return
statements within this function.
33da498
to
a4e6039
Compare
a4e6039
to
d48958b
Compare
d48958b
to
98064f6
Compare
98064f6
to
01dfdea
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #24 +/- ##
==========================================
+ Coverage 73.31% 73.87% +0.55%
==========================================
Files 80 81 +1
Lines 1889 1952 +63
Branches 274 281 +7
==========================================
+ Hits 1385 1442 +57
- Misses 493 499 +6
Partials 11 11 ☔ View full report in Codecov by Sentry. |
01dfdea
to
cd93693
Compare
cd93693
to
5fef1ef
Compare
5fef1ef
to
9b8e0b2
Compare
9b8e0b2
to
6574022
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
6574022
to
7e9df23
Compare
7e9df23
to
dcdbc7c
Compare
dcdbc7c
to
12b7caa
Compare
12b7caa
to
36a1971
Compare
36a1971
to
a61b068
Compare
a61b068
to
b6d38c6
Compare
b6d38c6
to
f61cf69
Compare
f61cf69
to
4df8890
Compare
- A buyer should see their orders and status - A seller should see their orders made on their products [Delivers #187900465]
4df8890
to
0ec37d6
Compare
Code Climate has analyzed commit 0ec37d6 and detected 1 issue on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
What does this PR do?
This PR allows seller and buyer should be see their orders.
Description of Task to be completed?
a buyer should be able to see order has purchased
a seller should be able to see orders has been made on his or her products
How should this be manually tested?
Clone the repository.
Checkout to the branch ft-order-management-#187900465
Run npm install to install dependencies.
Run npm run dev to start the development server and login
navigate to /orders if you're a buyer
navigate to /dashboard/orders if you're a seller
Login and get your orders
Congratulation you have successfully seen your orders status
What are the relevant pivotal tracker stories?
[Delivers #187900465]