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

Support for relative links in the Title and in Body. Update FeedFetcher.php #2100

Closed
wants to merge 2 commits into from

Conversation

IgorA100
Copy link
Contributor

@IgorA100 IgorA100 commented Feb 13, 2023

Support for relative links in the Title and in Body.

Signed-off-by: IgorA100 igora100@gmail.com

Summary

Checklist

Support for relative links in the Title and in Body.

Signed-off-by: IgorA100 <igora100@gmail.com>
@IgorA100 IgorA100 changed the title Update FeedFetcher.php Support for relative links in the Title and in Body. Update FeedFetcher.php Feb 13, 2023
@SMillerDev
Copy link
Contributor

This should really be fixed in feed.io, allowing us to request a full URL.

Signed-off-by: IgorA100 <igora100@gmail.com>
IgorA100 added a commit to IgorA100/news that referenced this pull request Feb 14, 2023
- Support for relative links in the Title and in Body. (nextcloud#2100

Signed-off-by: IgorA100 <igora100@gmail.com>
@IgorA100
Copy link
Contributor Author

This should really be fixed in feed.io

Am I making corrections in the wrong place?

@anoymouserver
Copy link
Contributor

Well, kind of, the feed parser used by NC News is debril/feed-io (currently using v4.x, but v5 is planned for the near future) and this kind of fix is best implemented there.

@IgorA100 IgorA100 marked this pull request as draft August 20, 2023 22:26
@IgorA100
Copy link
Contributor Author

IgorA100 commented Oct 16, 2023

I created a PR alexdebril/feed-io#419, but I am alarmed that there have been no new commits for almost a year...

My PR in main is approved.
I created PR in 5.3.x. alexdebril/feed-io#422
The problem will probably be solved soon!

@IgorA100
Copy link
Contributor Author

I close it as completed:
alexdebril/feed-io#422

@IgorA100 IgorA100 closed this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Relative paths in RSS feed links
3 participants