Skip to content

fix: Replace empty user name with user email#16176

Merged
PeerRich merged 1 commit intocalcom:mainfrom
shreya0204:fix-16171-no-name-dropdown
Aug 13, 2024
Merged

fix: Replace empty user name with user email#16176
PeerRich merged 1 commit intocalcom:mainfrom
shreya0204:fix-16171-no-name-dropdown

Conversation

@shreya0204
Copy link
Copy Markdown
Contributor

What does this PR do?

Loom Video: https://www.loom.com/share/41979dfe26444ed696ef6b8e847b2bb9?sid=8909d815-6a33-4352-9470-5f785b18a4c3

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have added a Docs issue here 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?

Please refer the above video.

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 12, 2024

@shreya0204 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 routing-forms area: routing forms, routing, forms ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Aug 12, 2024
@dosubot dosubot Bot added ui area: UI, frontend, button, form, input 🐛 bug Something isn't working labels Aug 12, 2024
@graphite-app graphite-app Bot added the community Created by Linear-GitHub Sync label Aug 12, 2024
@graphite-app graphite-app Bot requested a review from a team August 12, 2024 18:35
Copy link
Copy Markdown
Contributor

@anikdhabal anikdhabal left a comment

Choose a reason for hiding this comment

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

👍

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Aug 12, 2024

Graphite Automations

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

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

"Add community label" took an action on this PR • (08/12/24)

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

@PeerRich PeerRich merged commit d5f7065 into calcom:main Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working community Created by Linear-GitHub Sync 🧹 Improvements Improvements to existing features. Mostly UX/UI ⚡ Quick Wins A collection of quick wins/quick fixes that are less than 30 minutes of work ready-for-e2e routing-forms area: routing forms, routing, forms ui area: UI, frontend, button, form, input

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-4137] Users with no name shown at the top of dropdown (routing forms)

3 participants