Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
4 tasks
christinayung opened this issue Jan 11, 2021 · 5 comments
Closed
4 tasks

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

christinayung opened this issue Jan 11, 2021 · 5 comments
Assignees
Labels

Comments

@christinayung
Copy link
Contributor

christinayung commented Jan 11, 2021

  • Update share feature on Repositories Page
    image

Loading icon keeps spinning

  • Update share feature on Advances Search

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

share_link_not_working

  • Update share from Project List Page
    image

  • Update from all Analysis type pages:
    image

@junjun-zhang
Copy link

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

@rosibaj rosibaj changed the title Sharing query by short URL is broken Remove the Share Button: Sharing query by short URL is broken Feb 8, 2021
@rosibaj rosibaj added this to the Donkey Kong Sprint 42 milestone Feb 8, 2021
@rosibaj rosibaj added the blocked blocked label Feb 8, 2021
@akachru-github akachru-github added this to the Donkey Kong Sprint 43 milestone Feb 24, 2021
@rosibaj rosibaj removed the blocked blocked label Feb 25, 2021
@rosibaj rosibaj changed the title Remove the Share Button: Sharing query by short URL is broken Update the Share Button: Sharing query by short URL is broken Feb 25, 2021
@rosibaj
Copy link
Contributor

rosibaj commented Mar 5, 2021

QA summary in #708

was able to generate short urls from all pages.

@rosibaj
Copy link
Contributor

rosibaj commented Mar 12, 2021

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

@justincorrigible
Copy link
Member

justincorrigible commented Mar 12, 2021

@rosibaj 'fraid so: #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
Copy link
Member

Closing as "not needed" before decomissioning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants