Ruslan Forostianov
Ruslan Forostianov
it would be nice to have this feature. +1
@alisman @haynescd @pieterlukasse I've changed code to provide different descriptions for static and dynamic Virtual Study by default. Check https://github.com/cBioPortal/cbioportal-frontend/pull/5016/commits/4b49f4fb01a94fb39bd9aea00bc0c74338ced210
This PR fixes this issue https://github.com/cBioPortal/cbioportal/issues/10758
remember to update the FAQ answer here https://docs.cbioportal.org/user-guide/faq/#is-it-necessary-to-log-in-to-use-virtual-studies-if-i-do-log-in-what-additional-functionality-do-i-gain
After fixing this issue, evaluate whether sanitization like this will become obsolete https://github.com/cBioPortal/cbioportal/commit/6d94d9b9de1ca30db9bc818971673ea4316b92be (not part of the main branch) Anonymous users must be unable to specify the `owner` and `users`...
> Hello @forus , can I work on this issue if no one else is? @Mehdi-17 Hello, sure. Please go ahead. Thank you!
This issue will likely become more common once we implement RFC83, making virtual studies available to all users.
> Why not the same behavior/message as what is now displayed when you somehow get a URL to a study that you don't have access to? That's what I see...
Here are the steps to remove session service settings for all users! 1. find out the container id of the session service database with `docker ps` 2. connect to the...
> Lastly just try to address as much as the sonar cloud issues as possible @haynescd please have another look. The last 2 sonar cloud issues are intended. There is...