cBioPortal / icebox

very low priority issues
0 stars 0 forks source link

Enhancements to Improved findability of Authorized Studies feature #468

Open jagnathan opened 1 year ago

jagnathan commented 1 year ago

Enhancement to Improved Findability of datasets feature.

https://www.thehyve.nl/articles/improved-findability-of-datasets-in-cbioportal

This feature is enabled or disabled by portal.properties field.

Screen Shot 2023-03-07 at 12 36 17 PM

User level setting for study user Add a checkbox to allow users to show only authorized studies "Show authorized studies only". By default, this should be checked.

Study level setting for study owner - to be implemented Provide a study level setting to show or hide the study to unauthorized users for requesting access on home page. If study file does not have this property, then it should default to not showing the study. Frontend and Backend change

Tasks

Files modified

jagnathan commented 1 year ago

Configure the default value of the checkbox in settings in Portal.properties. By default checked or unchecked for "Show authorized studies only".

jagnathan commented 1 year ago

AuthorizedStudies

jagnathan commented 1 year ago

Screen Shot 2023-01-27 at 12 44 33 PM

Current implementation and its behavior.

When the checkbox is selected:

Screen Shot 2023-01-27 at 12 46 50 PM

jagnathan commented 1 year ago

Suggestions for improvement by Ino and Aaron: "Show authorized studies only" changes the number of studies available for selection. So the checkbox "Select all listed studies (3)" should be below instead of above.

When the "Show authorized studies only" is checked:

The number of available studies will be 2. The Left side menu should not show CNS/Brain and the secion for CNS/Brain should not be visible in the main section.

inodb commented 1 year ago

@jagnathan

Let's actually put the authorized selection here:

Screen Shot 2023-02-01 at 10 42 42 AM

Can use "Authorized: yes/no"

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 47 14 PM With options for both genome and authorization

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 47 22 PM Authorized is false - Shows studies without access

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 47 36 PM Authorized is true - Shows studies with access

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 47 56 PM Error message when trying to select study in not authorized

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 47 44 PM No error message when trying to select study is authorized

jagnathan commented 1 year ago

Screen Shot 2023-03-01 at 12 49 04 PM Screen Shot 2023-03-01 at 12 49 14 PM Two filters at the same time.

jagnathan commented 1 year ago

Some comments are hidden as they have outdated requirements.