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

fix: /settings/treams page layout issue #12947 #12974

Closed

Conversation

Rksingh090
Copy link

Fixes #12947

Copy link

vercel bot commented Jan 1, 2024

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

A member of the Team first needs to authorize it.

Copy link
Contributor

github-actions bot commented Jan 1, 2024

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

@github-actions github-actions bot added Low priority Created by Linear-GitHub Sync 🤔 can't reproduce The team was unable to reproduce this. Please provide a loom.com or birdeatsbugs.com video labels Jan 1, 2024
Copy link
Contributor

github-actions bot commented Jan 1, 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! 🙌

Comment on lines +78 to +79
const locale =
session?.data?.user.locale ?? typeof window !== "undefined" ? window.document.documentElement.lang : "en";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why no use of props.pageProps.newLocale ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what's that?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rksingh090 Rksingh090 changed the title Fix: /settings/treams page layout issue #12947 fix: /settings/treams page layout issue #12947 Jan 1, 2024
@PeerRich PeerRich closed this Jan 2, 2024
@Rksingh090 Rksingh090 deleted the 12947-teams-page-layout-issue branch January 13, 2024 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤔 can't reproduce The team was unable to reproduce this. Please provide a loom.com or birdeatsbugs.com video Low priority Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/settings/teams layout is wrong
3 participants