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

[Bug]: Invalid URL Created When Changing Pages On Topic List #96

Closed
TheNetNomad opened this issue May 8, 2023 · 2 comments
Closed

[Bug]: Invalid URL Created When Changing Pages On Topic List #96

TheNetNomad opened this issue May 8, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@TheNetNomad
Copy link

Steps to reproduce

  1. Go to https://lemmybb.rollenspiel.monster/view_forum?f=12
  2. In the upper right-hand corner, below the Login and Registration links, click either the "2" button or the ">" button to advance to the next page
  3. This should take you to a 404 screen. Find viewforum in the resulting url, change it to view_forum, and you should arrive at the intended destination

Reproduced in Firefox, Chrome and Edge on Windows 10 on rollspiel.monster and on a private instance

Version

0.2.2

LemmyBB Logs

No response

Additional context

Doing a quick search in the repo turns up src/routes/community.rs line 53:

let pagination = Pagination::new(page, limit, format!("/viewforum?f={f}&"));

Changing viewforum to view_forum there and recompiling seems to have fixed it on my own instance

@TheNetNomad TheNetNomad added the bug Something isn't working label May 8, 2023
@TheNetNomad TheNetNomad reopened this May 8, 2023
@TheNetNomad
Copy link
Author

This is probably related to issue 94 (I had already half-written the ticket when I found that pagination line, otherwise I would have commented this there, sorry!) but notably both the rollenspiel category used as an example and my private category are not on remote instances

@Nutomic
Copy link
Member

Nutomic commented May 10, 2023

Good explanation, with this I found the problem which was simply a typo. Turns out #94 was about the same thing but I completely misunderstood it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants