Skip to content

fix: API Session Type Errors#13545

Merged
joeauyeung merged 2 commits intomainfrom
api-fix-session-type-error
Feb 5, 2024
Merged

fix: API Session Type Errors#13545
joeauyeung merged 2 commits intomainfrom
api-fix-session-type-error

Conversation

@joeauyeung
Copy link
Copy Markdown
Contributor

What does this PR do?

The upId and profile is used on the web app for profile switching. Since the API relies on API keys, we can set this to null as done in apps/web/pages/api/link.tsapps/web/pages/api/link.ts

Fixes # (issue)

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

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • API should build

Mandatory Tasks

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

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 5, 2024

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

Name Status Preview Comments Updated (UTC)
ai 🛑 Canceled (Inspect) Feb 5, 2024 6:34pm
api 🛑 Canceled (Inspect) Feb 5, 2024 6:34pm
cal-demo 🔄 Building (Inspect) Visit Preview 💬 Add feedback Feb 5, 2024 6:34pm
4 Ignored Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2024 6:34pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2024 6:34pm
qa ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2024 6:34pm
ui ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2024 6:34pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 5, 2024

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

@keithwillcode keithwillcode added consumer core area: core, team members only labels Feb 5, 2024
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 5, 2024

📦 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! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Feb 5, 2024

No failed tests 🎉

@joeauyeung joeauyeung merged commit fde3b49 into main Feb 5, 2024
@joeauyeung joeauyeung deleted the api-fix-session-type-error branch February 5, 2024 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consumer core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants