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

A querystring appearing in the URL after accessing a page with a listing/search block #4692

Closed
sneridagh opened this issue Apr 13, 2023 · 7 comments

Comments

@sneridagh
Copy link
Member

After merging #4159, the site URL start to have a strange behavior, if a listing block is present, then the URL shows a querystring:

image

The interesting thing is that it appears everywhere once you click in another router link.

In the videos in #4159 it does not show, maybe something that was introduced lately in the PR?

/cc @bipoza @erral @ionlizarazu this is quite urgent, we would need to fix asap. If it's not possible, I'd better revert the change and proceed with another release.

@sneridagh
Copy link
Member Author

Also, I can see that the @querystring endpoint is being hit multiple times... this is another thing that we have to make sure that doesn't happen.

@erral
Copy link
Member

erral commented Apr 13, 2023

We will look at it

@wesleybl
Copy link
Member

wesleybl commented Apr 13, 2023

Also, I can see that the @querystring endpoint is being hit multiple times... this is another thing that we have to make sure that doesn't happen.

Maybe that's not just in this PR. See:

#4124

@sneridagh
Copy link
Member Author

sneridagh commented Apr 13, 2023

@erral Thanks a lot! Let's see if we can push a fix asap! If not, let me know and we can revert and postpone it until we have a better one. Please keep me posted!

@erral
Copy link
Member

erral commented Apr 14, 2023

It looks like the code under the useEffect in usePagination is run no matter the user has gone away from the page where the pagination is shown.

Somehow this useEffect should be deactivated (sorry if the concept is not correct) when browsing away.

@sneridagh
Copy link
Member Author

@erral we just merged some fixes to the listing/search blocks in cases of two of them are present in the same page, see:

#4694

Maybe now the situation is improved.

It's what I saw, the usePagination hook should be addressed.

@erral
Copy link
Member

erral commented Apr 14, 2023

I managed to remove the querystring before this merge, so I will test it again.

Anyway I have seen something strange going on, because I removed the querystring when going to any page except the home page. I need to test my fix further.

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

No branches or pull requests

4 participants