-
Notifications
You must be signed in to change notification settings - Fork 2
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
Homepage cms #2029
Homepage cms #2029
Conversation
@ardeay This is ready, tested on both, prod and stage environment |
Sync and Tested with next13 |
supportingText: content?.security_feature_supporting_text, | ||
image: content?.security_feature_image?.data?.[0]?.url, | ||
}; | ||
|
||
return ( | ||
<> | ||
<ThemeProvider theme={() => revampTheme(palette.mode)}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 themeproviders ?
width={'100%'} | ||
height={'100%'} | ||
/> | ||
<img src={heroImage} alt={generateAlt('Zesty image')} loading="eager" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
widht and ht?
@@ -74,6 +74,11 @@ const GetDemoSection = dynamic(() => import('revamp/ui/GetDemoSection'), { | |||
loading: () => <p>Loading...</p>, | |||
}); | |||
|
|||
export const removeHTMLtags = (str) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move this in utils file we can use it later
Description
Connect the homepage back to CMS
Fixes # (issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list the simulator and android emulators used.