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

Add extra NZ feeds + fix media data for The Conversation #67

Merged
merged 3 commits into from
Jan 28, 2025

Conversation

marcus-crane
Copy link
Contributor

@marcus-crane marcus-crane commented Jan 28, 2025

This PR adds a number of NZ feeds, many of which are ultimately owned by NZME or Stuff Limited but it should flesh out a lot of local coverage.

I also realised that The Conversation is technically an Australian publication.

In the same vein as the discussion around subpaths, The Conversation uses subpaths for "editions" so eg; theconversation.com/nz covers NZ and the /nz subpath is present in the NZ Edition RSS feed.

As you'll spot, subpaths are also at play for many of the NZME (nzherald) feeds which appear on the web as distinct publications while clearly using the same CMS and website as the actual NZ Herald.

I'll be following up with media data but I'll need to see what (if any) changes are needed in the UI to properly support subpaths as mentioned in Discord.

Signed-off-by: Marcus Crane <marcus@utf9k.net>
Signed-off-by: Marcus Crane <marcus@utf9k.net>
Signed-off-by: Marcus Crane <marcus@utf9k.net>
@vprelovac vprelovac merged commit f0b5299 into kagisearch:main Jan 28, 2025
1 check passed
@vprelovac
Copy link
Contributor

Great work

@marcus-crane
Copy link
Contributor Author

marcus-crane commented Jan 28, 2025

It's quite fun detective work haha. Te Ao News doesn't even surface an RSS feed but I realised their CMS is the same as NZ Herald (from poking around their sitemap) and got it to expose one by cross-referencing the CMS docs 🤓

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.

2 participants