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

sign up issues if user enters enter multiple spaces in first name #2358 #119

Closed
wants to merge 4 commits into from

Conversation

0mahendra
Copy link
Contributor

sign up issues if user enters enter multiple spaces in first name
#2358

1.Enter new email id in email field
2.Click on continue
3.Enter multiple spaces and then user first name
4.Fill up remaining information
5.Verify OTP
Bug 1-
Expected-
The user's first name should be present in the verification mail
Actual-
First name is not present as user provided spaces in first name

Bug 2
Expected-
team name should come properly after sign up
Actual-
Team name only contain '

https://techdome-my.sharepoint.com/:v:/g/personal/shreyas_pharande_techdome_net_in/ETjqjEQw3PFOpVK8DPQ-oYoBAPmf8vRCY_NEbMUhzejUxw?e=8jr9WI

after solving the bug

https://techdome-my.sharepoint.com/:v:/g/personal/mahendra_singh_techdome_net_in/EannLDQWUSNKu9GGw6pRdq4BFFiQIwkfBX2JJeILR-RBUQ?nav=eyJyZWZlcnJhbEluZm8iOnsicmVmZXJyYWxBcHAiOiJTdHJlYW1XZWJBcHAiLCJyZWZlcnJhbFZpZXciOiJTaGFyZURpYWxvZy1MaW5rIiwicmVmZXJyYWxBcHBQbGF0Zm9ybSI6IldlYiIsInJlZmVycmFsTW9kZSI6InZpZXcifX0%3D&e=ANN35n

@0mahendra 0mahendra added the bug Something isn't working label Jan 9, 2025
@0mahendra 0mahendra self-assigned this Jan 9, 2025
Copy link
Member

@Astitva877 Astitva877 left a comment

Choose a reason for hiding this comment

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

@0mahendra you have pushed the same code in two different PRs, whenever you make a new branch make it from development not from existing PR.

@@ -81,7 +81,7 @@
try {
const magicCodeResponse = await sendMagicCodeEmail({ email });
if (magicCodeResponse.isSuccessful) {
navigate(`/verify-magic-code/${email}`); // Updated this line
navigate(`/verify-magic-code/${email}/${magicCodeResponse.data}`); // Updated this line
Copy link
Member

Choose a reason for hiding this comment

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

why we have this changes in this PR also, i think you have already raised a PR for this.

{:else if authenticationError}
<small class="form-text text-dangerColor"
<small class="form-text text-dangerColor email_and_password_error"
Copy link
Member

Choose a reason for hiding this comment

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

also remove this, it has been already fixed in different branch.

@0mahendra 0mahendra closed this Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants