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

Remove ajax parameter when modifying the request #4743

Merged
merged 1 commit into from
Oct 18, 2022

Conversation

aledeg
Copy link
Member

@aledeg aledeg commented Oct 18, 2022

Closes #4726

Changes proposed in this pull request:

  • Fix author filter when clicking on article author link

How to test the feature manually:

  1. Check the related issue for steps to reproduce
  2. It should work properly.

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

Before, when clicking on the author name to use as a filter, the displayed page was a result page from the ajax query. This was due to the use of the ajax parameter in the query.
Now, the ajax parameter is removed from the query before displaying the filters.

Before, when clicking on the author name to use as a filter, the displayed page
was a result page from the ajax query. This was due to the use of the ajax
parameter in the query.
Now, the ajax parameter is removed from the query before displaying the filters.

Fix FreshRSS#4726
@aledeg aledeg added this to the 1.20.1 milestone Oct 18, 2022
@Alkarex Alkarex merged commit 0fd6084 into FreshRSS:edge Oct 18, 2022
@aledeg aledeg deleted the fix/ajax-links branch October 18, 2022 21:27
math-GH pushed a commit to math-GH/FreshRSS that referenced this pull request Nov 15, 2022
Before, when clicking on the author name to use as a filter, the displayed page
was a result page from the ajax query. This was due to the use of the ajax
parameter in the query.
Now, the ajax parameter is removed from the query before displaying the filters.

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

Successfully merging this pull request may close these issues.

[BUG] Opening certain links brings you to a broken page
2 participants