Skip to content

Change links to add feeds#3650

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
aledeg:hotfix/add-feed-link
May 31, 2021
Merged

Change links to add feeds#3650
Alkarex merged 1 commit intoFreshRSS:edgefrom
aledeg:hotfix/add-feed-link

Conversation

@aledeg
Copy link
Copy Markdown
Member

@aledeg aledeg commented May 31, 2021

Closes #3642

Changes proposed in this pull request:

  • Fix links to add feeds when hitting an empty page

How to test the feature manually:

  1. Make a search with no result to get an empty page
  2. Click the link to add some feeds
  3. Validate that you are redirected to the add feed form

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, the links was redirecting to the subscription management page which
was the default behavior before changes introduced in 1.17.0 (#3027). All
links were modified except the one for empty content.
Now, the empty content links are redirecting to the proper page.

Before, the links was redirecting to the subscription management page which
was the default behavior before changes introduced in 1.17.0 (FreshRSS#3027). All
links were modified except the one for empty content.
Now, the empty content links are redirecting to the proper page.

See FreshRSS#3642
@aledeg aledeg added this to the 1.18.1 milestone May 31, 2021
@aledeg
Copy link
Copy Markdown
Member Author

aledeg commented May 31, 2021

@mathContao

@aledeg aledeg added the UI 🎨 User Interfaces label May 31, 2021
@Alkarex Alkarex merged commit 4823e6d into FreshRSS:edge May 31, 2021
@aledeg aledeg deleted the hotfix/add-feed-link branch May 31, 2021 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI 🎨 User Interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] add some feeds -> link to wrong page

2 participants