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

Purchase History and Purchase Details pages #49

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Conversation

aaronkim218
Copy link
Contributor

@aaronkim218 aaronkim218 commented Dec 8, 2023

Description

Includes Purchase History and Purchase Details pages (only for designers). Nav dropdown contains Purchase History link that leads to Purchase History page (link only displays if current user is a designer). Purchase History page lists all of a designer's jobs, with most recent jobs displaying first. Clicking a job leads to Purchase Details page which shows details like shipping address, tracking number, price... Purchase Details page styled exactly like figma

Added StyledAvatar component, and changed any existing Avatars to StyledAvatar.

Added tracking number and estimated delivery date to job schema in frontend and backend

Changed location field of an address to match how it is stored in backend

How Has This Been Tested?

Tested locally

Checklist

  • I have performed a self-review of my code
  • I have reached out to another developer to review my code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing unit tests pass locally with my changes
Screenshot 2023-12-07 at 9 12 22 PM Screenshot 2023-12-07 at 9 12 30 PM Screenshot 2023-12-07 at 9 27 48 PM Screenshot 2023-12-07 at 9 33 43 PM

@aaronkim218 aaronkim218 changed the title PurchaseHistory and PurchaseDetails pages Purchase History and Purchase Details pages Dec 8, 2023
@alderwhiteford
Copy link
Contributor

Lets discuss before merging

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

Successfully merging this pull request may close these issues.

2 participants