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

[BUG] After inviting the user, email ID is copied to Name field as well #595

Closed
Saket2 opened this issue Sep 19, 2020 · 2 comments · Fixed by #2787
Closed

[BUG] After inviting the user, email ID is copied to Name field as well #595

Saket2 opened this issue Sep 19, 2020 · 2 comments · Fixed by #2787
Labels
Admin Settings Issues in admin settings pages Bug Something isn't working Good First Issue Good for newcomers hacktoberfest All issues that can be solved by the community during Hacktoberfest Low An issue that is neither critical nor breaks a user flow Team Managers Pod Issues that team managers care about for the security and efficiency of their teams Verified When issue is retested post its fixed

Comments

@Saket2
Copy link
Contributor

Saket2 commented Sep 19, 2020

Issue: After inviting the user, email ID is copied to Name field as well. Either Name should be left blank or it should be updated later on.

Browser Details: Chrome Version 85.0.4183.102 (Official Build) (64-bit)
Device Details: macOS 10.15.6 (MacBook Pro)
Note: Tried on cloud.

Steps to replicate :
1.Login to https://app.appsmith.com/user/login
2. go to 'Manage users'.
3. Click on 'Invite Users'
4. Fill up the details and click on 'invite'.

Actual Result: 'email ID' provided is also copied to the Name field.
Expected Result: email ID provided should not be copied to the Name field.

Attachment: https://drive.google.com/drive/folders/1dP-YRPvK_9zalv9qAKOQjs7hM8xob6HW?usp=sharing

@Saket2 Saket2 added the Bug Something isn't working label Sep 19, 2020
@Nikhil-Nandagopal Nikhil-Nandagopal added Admin Settings Issues in admin settings pages above-them labels Sep 21, 2020
@Nikhil-Nandagopal Nikhil-Nandagopal added the Low An issue that is neither critical nor breaks a user flow label Sep 30, 2020
@Nikhil-Nandagopal Nikhil-Nandagopal added hacktoberfest All issues that can be solved by the community during Hacktoberfest Good First Issue Good for newcomers labels Oct 19, 2020
@trdillon
Copy link
Contributor

I would like to work on this issue if it is still available. I would just like to clarify the scope of this so I know what direction to work in. Should this be addressed by just preventing the copying of the Email into the Name field and leaving it blank? Or should the functionality to edit the user's Name be added as well?

@Nikhil-Nandagopal
Copy link
Contributor

@trdillon go for it! I think for now just preventing the copy should be great, once we have an additional page for account setting, this makes sense

@close-label close-label bot added the QA Needs QA attention label Feb 1, 2021
@prapullac prapullac added Verified When issue is retested post its fixed and removed QA Needs QA attention labels Feb 5, 2021
@github-actions github-actions bot added the Team Managers Pod Issues that team managers care about for the security and efficiency of their teams label Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Settings Issues in admin settings pages Bug Something isn't working Good First Issue Good for newcomers hacktoberfest All issues that can be solved by the community during Hacktoberfest Low An issue that is neither critical nor breaks a user flow Team Managers Pod Issues that team managers care about for the security and efficiency of their teams Verified When issue is retested post its fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants