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

Deleted users still appear in the list and can be deleted again #84

Open
acornestean opened this issue Mar 31, 2022 · 5 comments
Open
Labels
classification Relating to classification analysis tool

Comments

@acornestean
Copy link
Collaborator

Description:
Adding users and then deleting them will initially no longer show them in the “Delete Users” list. However, adding another user will cause the first added users, which should be deleted, still appear in the “Delete Users” list and can be deleted again – with the confirmation message appearing.

Steps to reproduce:

  1. Access http://94.3.209.235:8502/
  2. Log in as admin
  3. Select “User Management” from the “Choose your operation” drop down.
  4. Add a user. The confirmation message is displayed.
  5. From the “Delete Users” list, check that the previously added user is listed there. The newly added user should be present in the list.
  6. From the “Delete Users” list, select the previously added user and delete it. A confirmation message is displayed.
  7. From the “Delete Users” list, check that the previously deleted user is no longer listed there. The deleted user should no longer be present in the list.
  8. Add another user
  9. Check the “Delete Users” list.
  10. Notice that the deleted user from Step 6 is listed there
  11. Selected the “deleted” user and delete it again
  12. Notice the user gets deleted again and a confirmation message is displayed.

Expected:
Deleted users should no longer be listed in the “Delete Users” list.

Actual:
Deleted users are re-listed in the “Delete Users” list after adding a new user. Furthermore, the “deleted” users can be deleted again.

2022-03-31_16h06_34.mp4
@acornestean acornestean added the classification Relating to classification analysis tool label Mar 31, 2022
@iRanadheer
Copy link
Contributor

Hey @acornestean,

We initially thought of adding the users from the admin account, but we added a signup feature for that, so, we are removing the add_users function from the 'user management' section. With that, I'm closing this issue, please re-open if needed.

@acornestean
Copy link
Collaborator Author

Issue is occurring again.

The only changes are in the way it is triggered now, results being the same. Currently, instead of adding a user as an admin to trigger the issue as before, I have to create a new user and the issue happens on the admin side.

@acornestean acornestean reopened this Apr 15, 2022
@iRanadheer
Copy link
Contributor

Try now?

@acornestean
Copy link
Collaborator Author

@iRanadheer

I can still reproduce the issue in the way I mentioned in my previous comment.

Basically, you delete a user and initially it’s gone from the list in Users Management. But if I open another instance of the app and create a new account, on the admin side the previously deleted user will re-appear in the list.

@iRanadheer
Copy link
Contributor

Okay, let's remain this open for now. This will not affect the RAs to label the data points. All of the tasks can be done outside the app. I will check why this problem keeps re-appearing.

@iRanadheer iRanadheer reopened this Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
classification Relating to classification analysis tool
Projects
None yet
Development

No branches or pull requests

2 participants