Skip to content

Conversation

@Joggz
Copy link

@Joggz Joggz commented Jan 5, 2020

No description provided.

Copy link

@John-pels John-pels left a comment

Choose a reason for hiding this comment

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

Add this to the page please

image

@Joggz
Copy link
Author

Joggz commented Jan 5, 2020

Add this to the page please

image

I will but I was hoping we should have that page with real data, instead of a static data..

@John-pels
Copy link

Add this to the page please
image

I will but I was hoping we should have that page with real data, instead of a static data..

Just add this component to the page and it would be easier to map the real data in it to make it dynamic.

Copy link

@Mlg29 Mlg29 left a comment

Choose a reason for hiding this comment

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

Pls look into what John said

Idris kindly look into your code
firstly, the add to cart button is not dere.

secondly, if u go through the page, the subscription page is seating untop of your footer probably because i use a position property in the component, i will advice you put the footer in a div den give the div a margin top. to make it look nice

@Joggz
Copy link
Author

Joggz commented Jan 6, 2020

Pls look into what John said

i have done that, you can pull and compare

@Joggz Joggz requested a review from John-pels January 6, 2020 11:21
Copy link

@Mlg29 Mlg29 left a comment

Choose a reason for hiding this comment

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

Nice one. but you can make the shopping cart dynamic. I believe the add to cart page has been merged, so when they click on it, it should update the shopping cart page

Copy link

@bamiogunfemi bamiogunfemi left a comment

Choose a reason for hiding this comment

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

2 feebacks
image
plus i think
image
this shoud selectively show item / items based on the items and not the (s)

@Joggz
Copy link
Author

Joggz commented Jan 8, 2020

2 feebacks
image
plus i think
image
this shoud selectively show item / items based on the items and not the (s)

I will look into this , when we start writing the cart functionality into the app

Copy link

@John-pels John-pels left a comment

Choose a reason for hiding this comment

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

Where is the Buy button?

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.

5 participants