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

#48 implement buyer order tracking #54

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

aimedivin
Copy link
Contributor

@aimedivin aimedivin commented Jul 5, 2024

What does this PR do?

This PR adds the order tracking feature for buyers. It shows the current status of each order and allows buyers to view all their orders and update the status of specific orders.

Description of Task to be completed?

Task Involved:

  • Display a list of all orders on the orders page.
  • Allow updating the status of a specific order.
  • Manage the state of all orders using Redux.
  • Style the orders list using Tailwind CSS.
  • Ensure the UI is responsive and looks good on desktop, tablet, and mobile devices.
  • Show a loading indicator while data is being fetched.
  • Implement an "Order Not Found" page for invalid order IDs.
  • Write unit tests for the Redux slice and the React components to ensure proper functionality.

How should this be manually tested?

  • Ensure You have backend service running and set VITE_APP_API_URL in your .env file.
  • Clone this repository and navigate to the project directory
    git clone -b ft-register https://github.com/atlp-rwanda/knights-ecomm-fe.git
    cd knights-ecomm-fe
    git checkout ft-buyer-orders
    npm i
    npm run dev

What are the relevant pivotal tracker/Trello stories?

#48

Screenshots

image
image
image

Copy link

netlify bot commented Jul 5, 2024

Deploy Preview for knights-fecommerce ready!

Name Link
🔨 Latest commit 8f5b1fe
🔍 Latest deploy log https://app.netlify.com/sites/knights-fecommerce/deploys/668bc3ee9653990008d5b6de
😎 Deploy Preview https://deploy-preview-54--knights-fecommerce.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jul 5, 2024

Deploy Preview for knight-fe ready!

Name Link
🔨 Latest commit 8f5b1fe
🔍 Latest deploy log https://app.netlify.com/sites/knight-fe/deploys/668bc3eee1b79f0008b33429
😎 Deploy Preview https://deploy-preview-54--knight-fe.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aimedivin aimedivin self-assigned this Jul 5, 2024
@aimedivin aimedivin added frontend Related to frontend in progress task has already been started labels Jul 5, 2024
@aimedivin aimedivin added this to the Cart, Wishlist and Checkout milestone Jul 5, 2024
@aimedivin aimedivin linked an issue Jul 5, 2024 that may be closed by this pull request
6 tasks
@aimedivin aimedivin force-pushed the ft-buyer-orders branch 3 times, most recently from dbe3a93 to cfb76ae Compare July 6, 2024 14:44
@coveralls
Copy link

Coverage Status

coverage: 84.444% (-3.9%) from 88.349%
when pulling cfb76ae on ft-buyer-orders
into 7ef982f on develop.

@aimedivin aimedivin force-pushed the ft-buyer-orders branch 2 times, most recently from 0300913 to 09f0da2 Compare July 7, 2024 11:09
@coveralls
Copy link

Coverage Status

coverage: 84.676% (-3.7%) from 88.349%
when pulling 09f0da2 on ft-buyer-orders
into 7ef982f on develop.

@coveralls
Copy link

Coverage Status

coverage: 84.633% (-3.7%) from 88.349%
when pulling 09f0da2 on ft-buyer-orders
into 7ef982f on develop.

@faid-terence
Copy link
Collaborator

@aimedivin and @UwicyezaG
Updates on this PR ?

@coveralls
Copy link

coveralls commented Jul 8, 2024

Coverage Status

coverage: 88.43% (+0.08%) from 88.352%
when pulling 8f5b1fe on ft-buyer-orders
into 97695aa on develop.

Implement Single Product View and Cart management (Guest and User)
@aimedivin
Copy link
Contributor Author

@aimedivin and @UwicyezaG Updates on this PR ?

@faid-terence It's ready for review.

Copy link
Collaborator

@faid-terence faid-terence left a comment

Choose a reason for hiding this comment

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

Nice Work @aimedivin and @UwicyezaG
PR Approved !

@faid-terence faid-terence merged commit e86110e into develop Jul 8, 2024
15 checks passed
Calebgisa72 pushed a commit that referenced this pull request Jul 8, 2024
Calebgisa72 pushed a commit that referenced this pull request Jul 8, 2024
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.

A buyer should be able to track their order status
7 participants