Skip to content
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

♻️ App Router migration - Consulting #3162

Closed
3 tasks done
jeoffreyfischer opened this issue Sep 25, 2024 · 3 comments · Fixed by #3209, #3219 or #3223
Closed
3 tasks done

♻️ App Router migration - Consulting #3162

jeoffreyfischer opened this issue Sep 25, 2024 · 3 comments · Fixed by #3209, #3219 or #3223
Assignees

Comments

@jeoffreyfischer
Copy link
Member

jeoffreyfischer commented Sep 25, 2024

Migrating Consulting pages to App Routing.

  • Consulting Index
  • Consulting pages
  • Consulting - Video Production
@github-project-automation github-project-automation bot moved this to 🤷‍♂️ Needs Triage in SSW.Website Sep 25, 2024
@jeoffreyfischer jeoffreyfischer moved this from 🤷‍♂️ Needs Triage to 🔖 Backlog - Ready in SSW.Website Oct 10, 2024
amankumarrr added a commit that referenced this issue Oct 11, 2024
Affected Route: `/consulting`

Fixed: #3162

Screenshot:
TODO
amankumarrr added a commit that referenced this issue Oct 15, 2024
Affected Route: `/consulting/video-production/*`

Fixed #3162

Screenshot:
TODO
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in SSW.Website Oct 15, 2024
@amankumarrr
Copy link
Member

image

Figure: Migrated consulting index page

@amankumarrr amankumarrr reopened this Oct 15, 2024
@github-project-automation github-project-automation bot moved this from ✅ Done to 🏗 In progress in SSW.Website Oct 15, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in SSW.Website Oct 16, 2024
@amankumarrr
Copy link
Member

Screenshot:
image

Figure: Migrated app routing video production page

@amankumarrr amankumarrr reopened this Oct 16, 2024
@github-project-automation github-project-automation bot moved this from ✅ Done to 🏗 In progress in SSW.Website Oct 16, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in SSW.Website Oct 17, 2024
@amankumarrr
Copy link
Member

Screenshot:
image
Figure: App Routing - Consulting/Artificial-intelligence

image
Figure: Before - Production slot

image
Figure: After - PR slot

amankumarrr added a commit that referenced this issue Oct 22, 2024
* App Routing - Consulting page

Affected Route: `/consulting`

Fixed: #3162

Screenshot:
TODO

* Adding animation for the consulting page

* Fixing the static path for the consulting pages

* Optimizing it for generic component

* removing unused prop from the server component

* Fixing Tina editing with the template

* Adding marketing props

* removing unused component

* removing client side rendering for tech cards

* Adding categories from the server side

* adding mediaCardsProps from server

* removing unused component import

* creating a generic component for props and tinaProp

* testing - removing useEffect for the animation

* Revert "testing - removing useEffect for the animation"

This reverts commit d63919b.

* Testing without prefetch

* Revert "Testing without prefetch"

This reverts commit 192616e.

* removing the additional font weight from the parent component

* Fixing the layout shifting issue with the banner

* fixing the placeholder for breadcrumb

* Adding video on the client size after the load

* Moving Video Embed on the client side

* loading the chatbot on the pure client side

* Revert "loading the chatbot on the pure client side"

This reverts commit 9197095.

* Adding video back to component

* Converting image types to WebP

* Converting import types to client to improve performance

* adding breadcrumb to dynamic import

* tree shaking for headlessUI library

* Revert "tree shaking for headlessUI library"

This reverts commit 6b84395.

* optimizing the package for headlessui

* optimizing ssw.megamenu and adding sourcemaps

* adding technologies card images on the client side

* Adding tailwind jid mode

* removing unused packages

* testing - prefetch for the homepage

* removing background video for the mobile devices

* Revert "testing - prefetch for the homepage"

This reverts commit dfb277f.

* removing unused imports

* Fixing the video loading on mobile

* Fixing the video on the mobile rendering

* Fixing the about us for mobile

* fixing the cumulative shift of carousel

* Adding placeholder image to avoid cumulative shift

* Adding prefetch logic for home page

* Changing the logic for mobile

* Revert "Adding placeholder image to avoid cumulative shift"

This reverts commit 866cc5d.

* reverting the changes for marketing component

* fixing the import path

* Adding the icon and image with dynamic

* Adding social icons to client side

* removing react icons from the regular import

* Fixing the issue with default import for social icon type

* adding twitter icon from dynamic import

* removing prefetch

* removing unused pathname import

* update pnpm-lock

* Update tina-lock.json

* reverting changes in social icons

* removing whitespace

* reverting the change for the azure and client logos

* adding client size side image rendering for azure banner

* reverting change for the consulting index

* Fixing server side rendering to reduce the load on client side

* removing dynamic importing for react-icon

* fixing the image import to server side

* removing breadcrumb from the home page

* Update tina-lock.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
2 participants