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] Cannot delete a user from the list #372

Closed
gpatruno opened this issue Jul 26, 2024 · 3 comments
Closed

[BUG] Cannot delete a user from the list #372

gpatruno opened this issue Jul 26, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@gpatruno
Copy link

Describe the bug
I'm trying to delete a user I've created. When I look at the documentation, it says :
To delete a user, navigate to /settings/users in your browser. Locate the Users table and click the action menu button (three dots) for the user you wish to delete.
However, I can't find the menu button for deleting a user.
I used the Docker image aaronleopold/stump:nightly, aaronleopold/stump:0.0.4 and aaronleopold/stump:0.0.3 and no one of these versions have a menu button in the users table.

To Reproduce
As administrator, create a user with rights. Still as admin, go to /settings/users to display the list of users.

As Server Owner:

  1. Go to '/settings/users'
  2. Click on 'Create User'
  3. reload the page '/settings/users'
  4. the new user appears in the list but without a menu

Expected behavior
I don't know if this feature is currently under development. I assume it should be present as it is documented.
Maybe I'm using the wrong Docker image or a wrong config... .

Screenshots
tableusers

Docker config:

services:

  stump:
    image: aaronleopold/stump:nightly
    container_name: stump
    volumes:
      - ./config:/config
      - ./media/books:/data
    ports:
      - 10801:10801
    environment:
      - PUID=1000
      - GUID=1000
      - STUMP_CONFIG_DIR=/config
    restart: unless-stopped

volumes:
  stump-volume:

Additional context
I've tried to use Postman to delete the user with his ID but I can't use the route. This may be a misuse, as I'm not familiar with the language used on the server side. However, I've retrieved the list of users by API with Postman.

@gpatruno gpatruno added the bug Something isn't working label Jul 26, 2024
@aaronleopold
Copy link
Collaborator

Hey 👋 thanks for reporting this!

I think this is just an instance where the UI needs to be tweaked a bit. The menu should be present but might be overflowing and require a horizontal scroll to access it. Can you confirm that the menu exists after attempting to scroll horizontally within the table?

@gpatruno
Copy link
Author

Hi, Thank you for your quick feedback.

Indeed, the button does exist in the table!
After adding the following css classes to the table tag, the menu appears.

display: block; overflow: auto;
Interface with this CSS :
image

do you think it would be possible to add a little styling to the table to make the scrollbar appear if not all the elements are displayed?

Otherwise thank you again for your feedback, have a good day !

@aaronleopold
Copy link
Collaborator

aaronleopold commented Jul 30, 2024

do you think it would be possible to add a little styling to the table to make the scrollbar appear if not all the elements are displayed?

Absolutely! I'll try to put up a quick fix this week to at least remove any intentional hiding of the scrollbar. We were also discussing implementing pinned columns on the Discord server, so certain columns (e.g. actionable things) are always visible and don't require scroll.

I'll close this issue out once the first, quicker fix is in the nightly image. It will be part of the next version (0.0.5) sometime after

Edit to add that I didn't have time to tackle this, but I'll try to get it out next week 👍

@JMicheli JMicheli changed the title [BUG] Cannot delete an user from the list [BUG] Cannot delete a user from the list Aug 5, 2024
@gpatruno gpatruno closed this as completed Aug 7, 2024
aaronleopold added a commit that referenced this issue Aug 18, 2024
aaronleopold added a commit that referenced this issue Aug 18, 2024
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

No branches or pull requests

2 participants