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

users: avoid create user resource #315

Merged
merged 1 commit into from
Oct 19, 2020
Merged

users: avoid create user resource #315

merged 1 commit into from
Oct 19, 2020

Conversation

sebdeleze
Copy link
Contributor

This commit is a rollback of functionality done before, to avoid user resource to be created when a user registers in public interface.

  • Avoids creating user resource when a user registers.
  • Hides Admin entry from user's menu, as this entry is already available in user's dropdown.
  • Shows invenio profiles entry in user's menu only if no user resource is associated to current user.
  • Changes dropdown display depending if user resource exists or not.
  • Closes Do not create user ressource when invenio account is created #314.

Co-Authored-by: Sébastien Délèze sebastien.deleze@rero.ch

@sebdeleze sebdeleze marked this pull request as ready for review September 21, 2020 14:52
@sebdeleze sebdeleze requested a review from zannkukai September 21, 2020 14:53
This commit is a rollback of functionality done before, to avoid user resource to be created when a user registers in public interface.

* Avoids creating user resource when a user registers.
* Hides `Admin` entry from user's menu, as this entry is already available in user's dropdown.
* Shows invenio profiles entry in user's menu only if no user resource is associated to current user.
* Changes dropdown display depending if user resource exists or not.
* Closes #314.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
@pronguen pronguen self-requested a review October 19, 2020 08:26
@sebdeleze sebdeleze merged commit 256fdb1 into rero:dev Oct 19, 2020
@sebdeleze sebdeleze deleted the sed-user-with-no-resource branch October 19, 2020 08:55
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 this pull request may close these issues.

Do not create user ressource when invenio account is created
3 participants