Skip to content
This repository has been archived by the owner on Jul 18, 2024. It is now read-only.

YSHOP2-92 : COD theme > Product Page > integrate the new design of variants, express checkout & reviews #211

Merged
merged 41 commits into from
Mar 3, 2023

Conversation

bj-anas
Copy link
Collaborator

@bj-anas bj-anas commented Feb 23, 2023

JIRA Ticket

YSHOP2-92

QA Steps

  • pnpm i
  • pnpm dev

Note

I didn't finish this page yet since my PR has many file changes and I don't want to make it harder for a reviewer to hand it also there are some backend bugs that need to be fixed until they are solved I will create another PR to complete all that's left.

@bj-anas bj-anas self-assigned this Feb 23, 2023
@bj-anas bj-anas added 💻 Ready for review Requires a review from another developer. and removed in progress labels Feb 28, 2023
assets/product.js Outdated Show resolved Hide resolved
assets/product.js Outdated Show resolved Hide resolved
assets/product.js Outdated Show resolved Hide resolved
assets/product.js Outdated Show resolved Hide resolved
@bj-anas bj-anas added 🛠 Requested changes Reviewer asked for some changes to be made. 💻 Ready for review Requires a review from another developer. and removed 💻 Ready for review Requires a review from another developer. 🛠 Requested changes Reviewer asked for some changes to be made. labels Mar 1, 2023
assets/product.js Outdated Show resolved Hide resolved
assets/product.js Outdated Show resolved Hide resolved
assets/product.js Outdated Show resolved Hide resolved
@bj-anas bj-anas merged commit f6c7b9c into main Mar 3, 2023
@bj-anas bj-anas deleted the DOT-7339 branch March 3, 2023 14:50
@bj-anas bj-anas changed the title COD theme > Product Page > integrate single product section part 2 YSHOP2-92 : COD theme > Product Page > integrate the new design of variants, express checkout & reviews Mar 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 Ready for review Requires a review from another developer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants