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

Shared link filtering and "Details" menus break the URL, removing the "/share" string #1038

Open
1 task done
pviprey opened this issue May 14, 2021 · 5 comments
Open
1 task done

Comments

@pviprey
Copy link

pviprey commented May 14, 2021

Prerequisites

  • I have searched open and closed issues to make sure that the bug has not yet been reported.

Bug report

This bug happens when using a private shared link. When clicking on 'DETAILS' on any windows for more details it updates the URL but removes the '/share' from it which will result in an infinite loop on the differents datas when you refresh the website/reopen the page after closing it earlier. Going back to the original page is also not enough as I still need to refresh it no matter what to get the datas.

Expected behavior
Load the datas whenever the site is resfresh or reopen later.

Environment (If applicable):

  • OS: Pop_OS
  • Browser: firefox
  • Version 88.0.1
@ukutaht
Copy link
Contributor

ukutaht commented May 17, 2021

Thanks for reporting! We'll take a look

@bz2
Copy link
Contributor

bz2 commented Aug 24, 2021

Same issue if applying any filters, the client-side history updating isn't preserving /share and that breaks the link.

@bz2
Copy link
Contributor

bz2 commented Aug 24, 2021

The pathname of the <Link/> components are wrong:

to={`/${encodeURIComponent(site.domain)}/${endpoint}${window.location.search}`}

to={{ pathname: `/${encodeURIComponent(site.domain)}/filter/${filterGroupForFilter(key)}`, search: window.location.search }}

Either need a bit passed in to represent share/not-share and include that, or have another abstraction that can correctly generate a base path.

@ukutaht
Copy link
Contributor

ukutaht commented Aug 25, 2021

All correct. Thanks @bz2

@RobertJoonas RobertJoonas changed the title opening Details breaks the load of the data if the page is reload Shared link filtering and "Details" menus break the URL, removing the "/share" string Sep 26, 2022
@cwaring
Copy link

cwaring commented Jun 13, 2023

Confirmed that I am also experiencing this issue.

Steps to recreate:

  1. Open a private shared link url
  2. Apply a filter using the custom ui overlay
  3. The /share/ path is removed from the url, thus breaking the url if it is copied and pasted to a new anonymous session

Notes:

  • Filters applied by clicking the links in pages list do not break the share url
  • If you add the /share/ path back into the url the functionality is restored

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

No branches or pull requests

4 participants