Final Products page. #5097
Final Products page. #5097
Conversation
Signed-off-by: Modern-artist <90668597+Modern-artist@users.noreply.github.com>
|
🚀 Preview for commit 112e4b1 at: https://6543202a8fec0a04d411d398--layer5.netlify.app |
|
Hey @Modern-artist Let's discuss this on website call tomorrow at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes. |
Signed-off-by: Modern-artist <90668597+Modern-artist@users.noreply.github.com>
Signed-off-by: Prem kumar <90668597+Modern-artist@users.noreply.github.com>
Signed-off-by: Prem kumar <90668597+Modern-artist@users.noreply.github.com>
|
🚀 Preview for commit 18c6c41 at: https://6548c4b886d5720cb35882e8--layer5.netlify.app |
|
Please add this as your agenda in Doc to present it in the Website Meet today at 6:30 pm IST/7:00 am CT |
Yeah @ankushbhardwaj408 // @Modern-artist |
|
🚀 Preview for commit a1173fe at: https://654bacd913188767397f3dbc--layer5.netlify.app |
|
|
@Modern-artist, it means that you have a bug. |
There was a problem hiding this comment.
These images should be colocated in the same components in which they are used.
There was a problem hiding this comment.
This SVG contains a number of PNGs and is not usable. Ensure only vector contents is used.
There was a problem hiding this comment.
This SVG contains a number of PNGs and is not usable. Ensure only vector contents is used.
There was a problem hiding this comment.
This image and other images here are really just PNGs and cannot be used.
There was a problem hiding this comment.
We already have this logo in this site's general assets. Reuse.
There was a problem hiding this comment.
We already have this logo in this site's general assets. Reuse.
src/pages/products/index.js
Outdated
There was a problem hiding this comment.
Like other pages, this page should be in sections and contain both sections and components.
There was a problem hiding this comment.
This SVG contains a number of PNGs and is not usable. Ensure only vector contents is used.
There was a problem hiding this comment.
This SVG contains a number of PNGs and is not usable. Ensure only vector contents is used.
Don't duplicate the logo.
There was a problem hiding this comment.
This SVG contains a number of PNGs and is not usable. Ensure only vector contents is used.
Don't duplicate.
src/pages/products/pricing.style.js
Outdated
| @@ -0,0 +1,569 @@ | |||
| import styled from "styled-components"; | |||
There was a problem hiding this comment.
A complete copy/paste? We're increasing our sustaining costs.
There was a problem hiding this comment.
No change, so don't include this file.
|
Most of the buttons included here don't work, @Modern-artist. This PR is not ready for review. |
|
The length of the CSS file needs cut down (by ~1/2) |
|
🚀 Preview for commit 449a447 at: https://654ef96c7dbdf35abc87e3ef--layer5.netlify.app |
|
Hey @Modern-artist Let's discuss this on website call tomorrow at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes. |
|
@Modern-artist are we pressing forward here or closing out and starting over? |
I would prefer to start again as I haven't written any code for the first two sections. |
Description
Final Products page.
This PR fixes #
Implemented Final Product page.
Notes for Reviewers
Signed commits