cnr-ibba/IMAGE-InjectTool

:recycle: The add an Existing AAP profile page is not clear

Opened this issue · 2 comments

bunop commented

Is your feature request related to a problem? Please describe.
It is not clearly stated that Add and existing AAP profile from accounts.viewsActivationComplete page requires to add an AAP profile already defined within the image manager account

Describe the solution you'd like
Having a custom AAP profile could be useful if you want to use the same user for manage biosample submission outside InjectTool. Ideally injectool need to ask for user credentials in order to verify that the user exists in BioSamples environment and for get the user_id. Then InjectTool create a new team using the manager user and add the user_id to the new Team (in other words, skip the user creation step, and add custom user to a new InjectTool managed team.

Describe alternatives you've considered
The adding AAP profile page could be removed, and accounts.views.ActivationComplete could redirect directly to the new BioSamples user generation. However an avanced user could want to have an unique account to manage his data within and outside InjectTool

Additional context

  • fix the biosample.views.RegisterUserView template title (is not a new account)
  • generate a token from user data (AAP username and password)
  • get user reference as described here or in pyUSIrest docs
  • create a new Team (like registering a new user does)
  • add the user reference to team
  • revise documentation accordingly
  • test stuff with unittests
  • test stuff in BioSamples test environment
  • what happens if I remove a submission from my data before injecttool does all stuff?
bunop commented

@SmartBioInf, do you want to add any comment?

@SmartBioInf, do you want to add any comment?

The solution described :

skip the user creation step, and add custom user to a new InjectTool managed team.

seems good to me