teddit
/
teddit
14
197
Fork 52
Code Issues 62 Pull Requests 2 Releases 2 Wiki Activity
Labels Milestones
New Issue

/search page drops sort and t parameters from next link #204

Open
opened 1 year ago by jbosboom · 1 comments
jbosboom commented 1 year ago

Steps to reproduce:

  1. Perform a search using non-default options for "sorted by:" and "links from:", bringing you to a page like https://teddit.net/r/baseball/search?q=baseball&restrict_sr=on&nsfw=on&sort=top&t=month .
  2. Scroll to the bottom of the page. Observe that the "next ›" link does not have a sort or t parameter, so clicking that link results in a page with no results.
  3. Manually edit the URL to add the missing parameters and click the link. (There are results.)
  4. Scroll to the bottom of the page. Observe that the "next ›" link is again missing the sort and t parameters.

(Similar to #194.)

Steps to reproduce: 1. Perform a search using non-default options for "sorted by:" and "links from:", bringing you to a page like https://teddit.net/r/baseball/search?q=baseball&restrict_sr=on&nsfw=on&sort=top&t=month . 2. Scroll to the bottom of the page. Observe that the "next ›" link does not have a `sort` or `t` parameter, so clicking that link results in a page with no results. 3. Manually edit the URL to add the missing parameters and click the link. (There are results.) 4. Scroll to the bottom of the page. Observe that the "next ›" link is again missing the `sort` and `t` parameters. (Similar to [#194](https://codeberg.org/teddit/teddit/issues/194).)
teddit added the
bug
 label 1 year ago
teddit commented 7 months ago
Owner

Now both prev and next link include sort and t parameters. 7c7709e74f The search functionality doesn't still work perfectly, especially when prev link is used, but it's better now.

Now both prev and next link include `sort` and `t` parameters. 7c7709e74f The search functionality doesn't still work perfectly, especially when prev link is used, but it's better now.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
Labels
Apply labels
Clear labels
bug
Something is not working duplicate
This issue or pull request already exists enhancement
New feature help wanted
Need some help invalid
Something is wrong question
More information is needed wontfix
This won't be fixed
No Label bug duplicate enhancement help wanted invalid question wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: teddit/teddit#204
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(<nil>)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes