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

Onboarding, input validation error isn't positioned correctly #4758

Merged
merged 6 commits into from
Oct 1, 2022

Conversation

gitstart
Copy link
Contributor

What does this PR do?

Error isn't properly placed.

191656610-dd1f99db-77c6-417d-96de-016d83c7cb4c

Fixes #4660

Environment: Staging(main branch) / Production

User Full Name Field:

Screen Shot 2022-09-23 at 16 01 31

User Bio Field:

Screen Shot 2022-09-23 at 16 02 17

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Chore (refactoring code, technical debt, workflow improvements)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Co-authored-by: gitstart <gitstart@gitstart.com>
Co-authored-by: gitstart <gitstart@users.noreply.github.com>
Co-authored-by: Nitesh Singh <nitesh.singh@gitstart.dev>
Co-authored-by: Matheus Muniz <87545749+matheusmuniz03@users.noreply.github.com>
Co-authored-by: Klinger Matheus <50892465+KlingerMatheus@users.noreply.github.com>
Co-authored-by: Eman <emmanuelgatwech@gmail.com>
Co-authored-by: Grace Nshokano <grace.devolop@gmail.com>
Co-authored-by: Olusanya Timothy <48022904+seunexplicit@users.noreply.github.com>
Co-authored-by: Thiago Nascimbeni <tnascimbeni@gmail.com>
@vercel
Copy link

vercel bot commented Sep 29, 2022

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

A member of the Team first needs to authorize it.

@gitstart
Copy link
Contributor Author

Hi @hariombalhara! Kindly review the PR

Copy link
Member

@PeerRich PeerRich left a comment

Choose a reason for hiding this comment

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

looks good, tho @hariombalhara mentioned the error should be part of the component

@hariombalhara
Copy link
Member

Good to go for now. Next time when we need to do similar fix it should go in component

@hariombalhara hariombalhara added ♻️ autoupdate tells kodiak to keep this branch up-to-date automerge labels Oct 1, 2022
@kodiakhq kodiakhq bot merged commit f3e2cf5 into calcom:main Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge ♻️ autoupdate tells kodiak to keep this branch up-to-date
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Onboarding, input validation error isn't positioned correctly
4 participants