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

Problem with users with role "user" #273

Closed
pronguen opened this issue Jul 29, 2020 · 1 comment · Fixed by #289
Closed

Problem with users with role "user" #273

pronguen opened this issue Jul 29, 2020 · 1 comment · Fixed by #289

Comments

@pronguen
Copy link
Contributor

  • when it is created from the admin interface, the user is attached to an org
  • when it is created from the public interface by the user himself/herself, the user is not attached to any org. In this case, it is not possible for an admin to give him/her another role like "submitter" because he/she cannot display this user.

2 possible solutions

  • when user is created from the public interface by the user himself/herself, it does not create a user resource, only an Invenio user
  • the admin is able to search and display users of his/her org + users not attached to an org
@sebdeleze sebdeleze changed the title Users with role "user", problem: Problem with users with role "user" Sep 2, 2020
@mmo
Copy link
Collaborator

mmo commented Sep 7, 2020

  • OK
  • Suggestion: Add a facet to user search, in order to filter users not attached to the organisation

sebdeleze pushed a commit that referenced this issue Sep 8, 2020
The admin users must be able to list users not associated to an organisation.

* Modifies the query for retrieving users to include users without organisation.
* Tests the existence of the organisation in user records to avoid an error if it not exists.
* Closes #273.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants