study view - bugs when adding/removing charts while filter is applied

[tmazor]

Go to MSK Impact study view: http://www.cbioportal.org/study?id=msk_impact_2017
Select Breast Cancer
Add "Metastatic Site" chart
--> problem 1: it adds as a pie chart, but should add as a table since there are >20 values
Click the "x" on the Metastatic Site pie chart to remove it.
Click "Clear All Filters"
--> problem 2: when the page reloads without the filter, the Metastatic Site chart appears

[jjgao]

@zhx828 I think both issues were reported. Could you link them?

[jjgao]

... and fix them :)

[zhx828]

Hm I can't reproduce the issue though. @jjgao can you?

[zhx828]

The first issue was solved a while ago.

[jjgao]

both seem fixed. I was able to reproduce a few hours ago... closing

[tmazor]

This is still happening for me - I've tested chrome & safari and both issues happen in both browsers

[jjgao]

hmm.. strange. I tried again and had the same issues. But after refreshing, the issues were gone.

[tmazor]

weird... even with a hard refresh, it happens consistently for me.

[jjgao]

hmm.. I am wondering if we have some inconsistency between the two servers. @inodb is deploying the new version. Let's try after that.

[tmazor]

I just tried and I still see both issues

[jjgao]

@tmazor could you try in another browser or in incognito? Trying to figure out if it's caching issue.

[tmazor]

I just tried incognito mode in chrome & safari and it happens in both

[jjgao]

This is very strange. @zhx828 any thoughts?

[zhx828]

@jjgao I was thinking the same thing. The latest code was not deployed to both servers. But at the time Tali tested yesterday 3pm. We have redeployed all wars to both servers. And the second issue on this ticket, it looks like a side effect of the first issue. But the thing is I tried both Chrome and Safari several times. I can't reproduce the issue.

[zhx828]

I now can reproduce the bug. Will fix it soon.

[zhx828]

@tmazor @jjgao this issue is now solved and deployed.