Skip to content

chore: migrate /apps/installation pages#18870

Merged
hbjORbj merged 3 commits intomainfrom
chore/migrate-apps-installation-pages
Jan 26, 2025
Merged

chore: migrate /apps/installation pages#18870
hbjORbj merged 3 commits intomainfrom
chore/migrate-apps-installation-pages

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Jan 24, 2025

What does this PR do?

  • Migrate /apps/installation pages to App Router

Tested

Screenshot 2025-01-26 at 4 44 19 PM

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

  • Test /apps/installation/[...step] pages

@graphite-app graphite-app Bot requested a review from a team January 24, 2025 15:04
@hbjORbj hbjORbj changed the title chore: migrate apps installation pages chore: migrate /apps/installation pages Jan 24, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2025 9:12pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Jan 26, 2025 9:12pm

@keithwillcode keithwillcode added consumer core area: core, team members only labels Jan 24, 2025
@dosubot dosubot Bot added the app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar label Jan 24, 2025
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jan 24, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (01/24/25)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 24, 2025

E2E results are ready!

@hbjORbj hbjORbj marked this pull request as ready for review January 26, 2025 21:04
@hbjORbj hbjORbj merged commit 2f9beed into main Jan 26, 2025
@hbjORbj hbjORbj deleted the chore/migrate-apps-installation-pages branch January 26, 2025 22:04
ThyMinimalDev pushed a commit that referenced this pull request Jan 27, 2025
* migrate apps/installation pages

* fix test
ThyMinimalDev added a commit that referenced this pull request Jan 27, 2025
* init router endpoint for v2

* fixup fixup

* resolve merge conflicts

* update libraries

* update router controller

* make router endpoint public

* chore: refactor router endpoint and add team/org id

* cleanup

* chore: Only import Sentry when needed (#18882)

* perf: prerender routes in main navigation (#18883)

* chore: migrate /apps/categories pages (#18871)

* migrate apps/categories pages

* remove pages dir

* chore: app router - /payment pages (#18152)

* remove Head from payment page comp

* remove route from pages router

* move out of future folder

* add to config.matcher

* remove unneeded code

* update middleware

* fix

* fix

* perf: replace `getServerSessionForAppDir` usages with our original `getServerSession` function (#18897)

* chore: migrate /apps/installation pages (#18870)

* migrate apps/installation pages

* fix test

* chore: migrate `/auth/logout` (#18901)

* migrate /auth/verify-email page

* update middleware

* migrate /auth/logout page

* expose router endpoint

* small refactor pageprops not necessary

* sentry is already tracking performance

* reuse logic

* fixup! reuse logic

* fixup! fixup! reuse logic

* fixup! fixup! fixup! reuse logic

---------

Co-authored-by: Morgan Vernay <morgan@cal.com>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
Co-authored-by: Benny Joo <sldisek783@gmail.com>
Co-authored-by: Morgan <33722304+ThyMinimalDev@users.noreply.github.com>
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
* migrate apps/installation pages

* fix test
MuhammadAimanSulaiman pushed a commit to hit-pay/cal.com that referenced this pull request Feb 25, 2025
* init router endpoint for v2

* fixup fixup

* resolve merge conflicts

* update libraries

* update router controller

* make router endpoint public

* chore: refactor router endpoint and add team/org id

* cleanup

* chore: Only import Sentry when needed (calcom#18882)

* perf: prerender routes in main navigation (calcom#18883)

* chore: migrate /apps/categories pages (calcom#18871)

* migrate apps/categories pages

* remove pages dir

* chore: app router - /payment pages (calcom#18152)

* remove Head from payment page comp

* remove route from pages router

* move out of future folder

* add to config.matcher

* remove unneeded code

* update middleware

* fix

* fix

* perf: replace `getServerSessionForAppDir` usages with our original `getServerSession` function (calcom#18897)

* chore: migrate /apps/installation pages (calcom#18870)

* migrate apps/installation pages

* fix test

* chore: migrate `/auth/logout` (calcom#18901)

* migrate /auth/verify-email page

* update middleware

* migrate /auth/logout page

* expose router endpoint

* small refactor pageprops not necessary

* sentry is already tracking performance

* reuse logic

* fixup! reuse logic

* fixup! fixup! reuse logic

* fixup! fixup! fixup! reuse logic

---------

Co-authored-by: Morgan Vernay <morgan@cal.com>
Co-authored-by: Alex van Andel <me@alexvanandel.com>
Co-authored-by: Benny Joo <sldisek783@gmail.com>
Co-authored-by: Morgan <33722304+ThyMinimalDev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar consumer core area: core, team members only ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants