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

Access request upgrade #2530

Open
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

julien-louis
Copy link
Collaborator

@julien-louis julien-louis commented Dec 4, 2024

discussions : #2415

  • click access request on welcome page
    • popup "do you have an account"
      • yes : login page
        • redirect to access request form
          • if already pending access request : info msg + redirect to study list
          • if already full access to study : info msg + redirect to study
      • no : account request page
  • role & contact are no more to be set by the user
    • role
      • role is set (as a free string) by the study admin when he send the invitation email
      • the user doesn't see it directly and doesn't have to set it in the account request form
      • the role is then forwarded in the account request and displayed in user details by a shanoir admin (when he grants actual access to this new account)
    • contact
      • contact name (firstname + lastname) is automatically set when emitting an invitation (it's the inviter name)
      • transparent for the user as well
      • forwarded as well and displayed in user's details when the shanoir admin grants actual access

@michaelkain michaelkain changed the title Access request updgrade Access request upgrade Dec 5, 2024
@julien-louis

This comment was marked as outdated.

@julien-louis julien-louis marked this pull request as ready for review December 16, 2024 13:49
@michaelkain michaelkain self-assigned this Jan 6, 2025
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.

2 participants