MIT-LCP / physionet-build

The new PhysioNet platform.
https://physionet.org/
BSD 3-Clause "New" or "Revised" License
56 stars 20 forks source link

Distinguish/Rename "credentialed" with "identity verified" #1690

Open kepaik opened 2 years ago

kepaik commented 2 years ago

Lots of confusion when the first part of credentialing (identity verification) is completed, but the training report hasn't been processed.

Credentialing = (identity verification) + (training certification)

Chrystinne commented 2 years ago

@tompollard Do we need another pull request for this change? Or may I add this change to the currently opened pull request (https://github.com/MIT-LCP/physionet-build/pull/1689)?

Chrystinne commented 2 years ago

@tompollard I've created a new pull request. It might be better, even when it is a minor fix.

tompollard commented 2 years ago

Credentialing = (identity verification) + (training certification)

@kepaik This means that being "credentialed" is dataset specific because we support multiple training types. Fine with me but I want to check whether this is what you intend?

kepaik commented 2 years ago

This has less to do with the backend, but the front end for the users.

We've gotten many emails where they are confused that they have been "credentialed", but they actually do not have the credentials to access teh database.

kepaik commented 2 years ago

5F30CF2C@5261FB04 04234563 png C96800E1@E5DA011A 04234563 png

kepaik commented 2 years ago

Maybe if we change it to "verified" for identity, and then "credentialed" for each database.

tompollard commented 2 years ago

Maybe if we change it to "verified" for identity, and then "credentialed" for each database.

Yeah, I like the idea of something like this.

Chrystinne commented 2 years ago

@kepaik I've renamed the column "Credentialed" to "Identity Verified". It's available at https://physionet.org/console/training/review/

kepaik commented 2 years ago

The column name is not the issue. It's the informational page and emails to the user.

https://physionet.org/settings/credentialing/

See above images for reference.

Chrystinne commented 2 years ago

We need to replace the current text with the following:

"To access certain datasets, we require your identity to be "verified". This means that you must formally submit your personal details for review, so that we can confirm your identity.

Certain datasets also require you to complete training and/or to sign a Data Use Agreement. You can find specific requirements in the "Files" section of the project description."

To raise a support request, please click here.

Your account has been successfully verified on Sept. 19, 2022, 9:15 p.m"

kepaik commented 2 years ago

Your account has been successfully verified on Sept. 19, 2022, 9:15 p.m

Underneath, if they do not have their training completed, in a red info box:

Your account cannot access restricted resources.

  • Your training is still under review or
  • Complete required training: CITI Data or Specimens Only Research
  • Submit your training here
kepaik commented 2 years ago

The email, "Your application for PhysioNet credentialing", should be adjusted as well.

Thank you for your interest in the PhysioNet Clinical Databases. We are pleased to say that your identity has been verified and your application for credentialed access has passed the first stage.

You will not able to access protected databases until you have agreed to the terms of usage and completed any required training. For example, you can access MIMIC-III by following the steps below:

tompollard commented 2 years ago

Good suggestions, thanks Ken. @Chrystinne please could you take a look at these?

Chrystinne commented 2 years ago

@tompollard Sure. I was already talking to @kepaik about the changes we need to make to solve this issue.

Chrystinne commented 2 years ago

We need to replace the current text with the following:

Thank you for your interest in the PhysioNet Clinical Databases. We are pleased to say that your application for credentialed access has been approved.

You will not be able to access protected databases until you have agreed to the terms of usage and completed any required training. For example, you can access MIMIC-III by following the steps below:

To check the status of your training or to submit training documents for review, please visit https://physionet.org/settings/training/.

Regards, The PhysioNet Team, MIT Laboratory for Computational Physiology Institute for Medical Engineering and Science, MIT, E25-505 77 Massachusetts Ave, Cambridge, MA 02139

Chrystinne commented 2 years ago

After these changes, we now have three possibilities for the Credentialing information:

1- Last training status is "Accepted":

Training status Accepted

2- Last training status is "Review":

Training status under review

1- Last training status is "Rejected" or the user hasn't submitted it yet:

Training status reject or not yet submitted

Right, @kepaik?

Chrystinne commented 2 years ago

@kepaik could you please have a look at the images to confirm so I can create the pull request?

kepaik commented 2 years ago

It's still not very clear, particularly when distinguishing when a user has had their identity verified. This whole issue is similar to the dynamic status boxes I requested a while ago, so I'll convert that into an issue to provide more guidance. #1692