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

#187900465: A buyer and seller should be manage orders #24

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

yvanddniyo
Copy link
Collaborator

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]

Copy link

vercel bot commented Jul 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eagles-ec-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2024 1:07pm

dispatch(updateOrderStatus({ orderId, status }));
};

return (
Copy link

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 (
Copy link

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.

@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2024

Codecov Report

Attention: Patch coverage is 52.94118% with 32 lines in your changes missing coverage. Please review.

Project coverage is 73.87%. Comparing base (ff6239d) to head (0ec37d6).

Files Patch % Lines
src/redux/reducers/ordersSlice.ts 66.66% 15 Missing ⚠️
src/redux/reducers/registerSlice.ts 11.76% 15 Missing ⚠️
src/components/cards/ProductCard.tsx 0.00% 1 Missing ⚠️
src/pages/RegisterUser.tsx 0.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@MugemaneBertin2001 MugemaneBertin2001 left a comment

Choose a reason for hiding this comment

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

LGTM

    - A buyer should see their orders and status
    - A seller should see their orders made on their products

[Delivers #187900465]
Copy link

codeclimate bot commented Jul 26, 2024

Code Climate has analyzed commit 0ec37d6 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

View more on Code Climate.

@teerenzo teerenzo merged commit a60a1e6 into dev Jul 26, 2024
7 checks passed
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.

6 participants