icgc-dcc / dcc-portal

Data portal for exploring and accessing data
https://dcc.icgc.org/
Other
15 stars 8 forks source link

Update the Share Button: Sharing query by short URL is broken #715

Closed christinayung closed 10 months ago

christinayung commented 3 years ago

Loading icon keeps spinning

When the "Share" button above the table is clicked, usually a shortened link appears, but it's stuck on "Loading":

share_link_not_working

junjun-zhang commented 3 years ago

found a couple of places with specs/implementations of url shortening service:

rosibaj commented 3 years ago

QA summary in https://github.com/icgc-dcc/dcc-portal/issues/708

was able to generate short urls from all pages.

rosibaj commented 3 years ago

@caravinci are you still waiting on a code review here?

justincorrigible commented 3 years ago

@rosibaj 'fraid so: https://github.com/icgc-dcc/dcc-portal/pull/723 noticed there was a bug in the portal, creating new urls, because of a bug in the old url shortener itself.

this takes care of that.

justincorrigible commented 10 months ago

Closing as "not needed" before decomissioning