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: org team page not found with uppercase letters #11737

Merged
merged 2 commits into from
Oct 19, 2023

Conversation

G3root
Copy link
Contributor

@G3root G3root commented Oct 6, 2023

What does this PR do?

Fixes #11719

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)

@vercel
Copy link

vercel bot commented Oct 6, 2023

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

Name Status Preview Comments Updated (UTC)
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 17, 2023 8:47pm

@vercel
Copy link

vercel bot commented Oct 6, 2023

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

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the organizations area: organizations, orgs label Oct 6, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

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

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2023

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

Copy link
Contributor

@Udit-takkar Udit-takkar left a comment

Choose a reason for hiding this comment

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

@G3root the changes looks fine.

Not sure if it's related but can you look into why i get 404 on some teams?

Screen.Recording.2023-10-09.at.8.55.11.PM.mov

@PeerRich PeerRich added the Low priority Created by Linear-GitHub Sync label Oct 11, 2023
@PeerRich
Copy link
Member

@G3root wanna follow up here: #12003 ?

@maige-app
Copy link

maige-app bot commented Oct 19, 2023

Here is your custom prompt:

Avoid using the Bounty label. Don't use the Invalid label. Every time you add the "✨ feature" label, also add the "🚨 needs approval" label, unless the author is PeerRich. Reply to 'can I work on this'-type comments with 'No need to ask to get assigned, just go ahead and start working on the issue and open a PR. Please note this issue may not have been reviewed by the core product team and they may close the issue and the PR!'. When reviewing PRs, ensure the position of the floating action button does not overlap with other UI elements.

Share feedback by commenting.

@G3root G3root deleted the fix-org-name-casing branch October 19, 2023 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Low priority Created by Linear-GitHub Sync organizations area: organizations, orgs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-2585] org team page not found with uppercase letters
3 participants