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

QA Invite Users #6486

Closed
4 of 5 tasks
cgardens opened this issue Sep 27, 2021 · 8 comments
Closed
4 of 5 tasks

QA Invite Users #6486

cgardens opened this issue Sep 27, 2021 · 8 comments
Assignees
Labels

Comments

@cgardens
Copy link
Contributor

cgardens commented Sep 27, 2021

This feature has never worked properly. We need to figure out its current state and get it to a working state before the launch.

QA Checklist

  • - invite to a workspace a user that does not yet have an Airbyte account.
  • - invite to a workspace a user that does not yet have an Airbyte account should alert that the user does not exist and they first must create an account.
  • - invite to a workspace a user that does already have an Airbyte account.
  • - should not be able to add a user multiple times (@tuliren to do this)
  • - hide role in UI when inviting a user
    • This field is now disabled.
@cgardens cgardens added the type/enhancement New feature or request label Sep 27, 2021
@cgardens cgardens added this to the Core 2021-09-29 (qa) milestone Sep 27, 2021
@cgardens
Copy link
Contributor Author

@tuliren
Copy link
Contributor

tuliren commented Sep 28, 2021

Here is my QA summary without knowing what is the expected user flow.

Invite to a workspace a user that does NOT yet have an Airbyte account.

Issues:

  • It is unclear if I must select a role or not.
    • This is probably not a required field, since I can click the submission button without selecting a role.
  • There is only an admin role. Should we also have a user role?
    • Since role is not required, I guess not selecting any role is equivalent to a user role. But it's better to make this explicit.
  • Once a role is selected, there is no way to unselect it.
    • This is a problem if not selecting a role is equivalent to a user role.

Screen Shot 2021-09-28 at 12 25 40

  • When adding a new user, I cannot submit the form. A warning only shows up in the browser console:
console.js:107 Warning: An unhandled error was caught from submitForm() Error: Object not found. User liren+user1@airbyte.io does not exist yet. Please sign-up manually and add to workspace once user is created
    at AirbyteRequestService.ts:75
    at l (runtime.js:63)
    at Generator._invoke (runtime.js:293)
    at Generator.next (runtime.js:118)
    at r (asyncToGenerator.js:3)
    at s (asyncToGenerator.js:25)

Invite to a workspace a user that does already have an Airbyte account.

Adding a registered user for the first time works.

Issues:

  • Users are automatically added as admin, even if admin role is not selected.
  • The same user can be added multiple times.

Screen Shot 2021-09-28 at 12 36 03

Additionally issues related to account management

  • The remove button is clickable but does not actually remove the user.
  • The remove button is clickable even for the currently logged in user.
    • Because this button currently does not work, so I cannot test what will happen if the last user is removed from the workspace.
    • A user probably should not be able to remove herself? Or at least the super admin, if there is such a role, should not be able to remove herself. Otherwise, all users can be removed from the workspace, and the workspace becomes a phantom one.

@jrhizor
Copy link
Contributor

jrhizor commented Sep 28, 2021

Which environment are you testing on (and which version of core/cloud)?

@tuliren
Copy link
Contributor

tuliren commented Sep 28, 2021

Which environment are you testing on (and which version of core/cloud)?

I tested on the production site, which does not show a version. Should I do it on the dev site?

@jrhizor
Copy link
Contributor

jrhizor commented Sep 29, 2021

Prod is good. I looked up the version of core and cloud from Kubernetes on the cloud console.

@ChristopheDuong
Copy link
Contributor

When adding a new user, I cannot submit the form. A warning only shows up in the browser console:

From past discussions, we had mentioned about a pop-up (not just a warning in console) when this happens as a workaround:

Screenshot 2021-09-29 at 10 22 04

@davinchia
Copy link
Contributor

QA for current prod deployment. Looks like it's working:

invited a user:
Screen Shot 2021-10-06 at 8 19 50 AM

verified email:
Screen Shot 2021-10-06 at 8 20 50 AM

logged in to new user and saw new workspace:
Screen Shot 2021-10-06 at 8 21 09 AM

revoke the user:

(disappears from inviter)
Screen Shot 2021-10-06 at 8 22 24 AM

(disappears from invitee)
Screen Shot 2021-10-06 at 8 22 28 AM

@davinchia
Copy link
Contributor

@cgardens this looks good on prod. think we can close it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants