fix code reviews

pancakeswap · Oct 17, 2024 · b03bccc · b03bccc
1 parent d876b13
commit b03bccc
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 19 deletions.
diff --git a/apps/web/src/components/PhishingWarningBanner/Step3.tsx b/apps/web/src/components/PhishingWarningBanner/Step3.tsx
@@ -5,18 +5,6 @@ import { VerticalDivider } from '@pancakeswap/widgets-internal'
 export const Step3 = () => {
   const { t } = useTranslation()
 
-  // const PlayNowAction = (
-  //   <LinkExternalAction
-  //     href="https://pancakeswap.finance/swap?utm_source=Website&utm_medium=homepage&utm_campaign=PCSX&utm_id=PCSX"
-  //     color="#280D5F"
-  //     externalIcon="arrowForward"
-  //   >
-  //     <Flex color="#280D5F" alignItems="center" style={{ whiteSpace: 'nowrap' }}>
-  //       {t('Swap Now')}
-  //     </Flex>
-  //   </LinkExternalAction>
-  // )
-
   return (
     <Flex mr={['6px']} flexDirection="row" alignItems="center">
       <Text bold as="span" color="#FFFFFF" fontSize={['12px', '12px', '14px']}>

diff --git a/apps/web/src/views/Home/components/Banners/PCSXBanner.tsx b/apps/web/src/views/Home/components/Banners/PCSXBanner.tsx
@@ -79,7 +79,6 @@ const bgXsVariant: GraphicDetail = {
 
 const Desc = () => {
   const { t } = useTranslation()
-  const { isMobile } = useMatchBreakpoints()
 
   return (
     <StyledBannerDesc>
@@ -91,7 +90,6 @@ const Desc = () => {
 export const PCSXBanner = () => {
   const { t } = useTranslation()
   const { isMobile, isSm, isDesktop } = useMatchBreakpoints()
-  console.log({ isMobile, isSm, isDesktop })
 
   const PlayNowAction = (
     <LinkExternalAction

diff --git a/apps/web/src/views/Home/components/Banners/hooks/useMultipleBannerConfig.tsx b/apps/web/src/views/Home/components/Banners/hooks/useMultipleBannerConfig.tsx
@@ -41,10 +41,6 @@ export const useMultipleBannerConfig = () => {
 
   return useMemo(() => {
     const NO_SHUFFLE_BANNERS: IBannerConfig[] = [
-      {
-        shouldRender: true,
-        banner: <PCSXBanner />,
-      },
       {
         shouldRender: true,
         banner: <TurkeyMeetupBanner />,
@@ -57,6 +53,10 @@ export const useMultipleBannerConfig = () => {
         shouldRender: isRenderUserBanner.shouldRender && !isRenderUserBanner.isEarningsBusdZero,
         banner: <UserBanner />,
       },
+      {
+        shouldRender: true,
+        banner: <PCSXBanner />,
+      },
       {
         shouldRender: true,
         banner: <TgPredictionBotBanner />,

diff --git a/packages/utils/user/phishingBanner.ts b/packages/utils/user/phishingBanner.ts
@@ -8,7 +8,6 @@ const hidePhishingBannerAtom = atom(
   (get) => {
     const now = dayjs()
     const last = dayjs(get(phishingBannerAtom)).add(1, 'day')
-    // const notPreview = process.env.NEXT_PUBLIC_VERCEL_ENV !== 'preview'
     return last && now.unix() > last.unix()
   },
   (_, set) => set(phishingBannerAtom, Date.now()),