cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

chore: [app-router-migration 34] migrate the "/auth/logout" and "/auth/signin" pages

Open r4zendev opened this issue 2 years ago • 4 comments

What does this PR do?

  • This PR migrates the /auth/logout and /auth/signin pages to the app directory (which runs under the App Router).
  • Using /future/auth/logout is available at all times and is supposed to log the user out as /auth/logout page does.
  • However signin page (or fragment) was used before should behave the same using the /future/auth/signin route.

Requirement/Documentation

  • If there is a requirement document, please, share it here.
  • If there is ab UI/UX design document, please, share it here.

Type of change

  • [x] Chore (refactoring code, technical debt, workflow improvements)
  • [x] New feature (non-breaking change which adds functionality)

How should this be tested?

  • Using /future/auth/logout is available at all times and is supposed to log the user out as /auth/logout page does.
  • However signin page (or fragment) was used before should behave the same using the /future/auth/signin route. It was not tested locally.

Mandatory Tasks

  • [x] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

r4zendev avatar Jan 12 '24 13:01 r4zendev

@r4zendev is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jan 12 '24 13:01 vercel[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

:white_check_mark: DmytroHryshyn
:white_check_mark: hbjORbj
:x: r4zendev
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jan 12 '24 13:01 CLAassistant

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

github-actions[bot] avatar Jan 12 '24 13:01 github-actions[bot]

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

github-actions[bot] avatar Jan 18 '24 16:01 github-actions[bot]