Add subtitle field to feed fields#291
Conversation
|
@OpenHands please add a comment in the code explaining this change |
|
I'm on it! rbren can track my progress at all-hands.dev |
|
I've added comments explaining the subtitle field change and pushed them to the PR branch. Here's a summary of the changes: Changes Made✅ Added comment in // The subtitle field is used in Atom feeds to provide a secondary description or tagline for the feed
'subtitle',✅ Added JSDoc comment in /** The subtitle field from Atom feeds, providing a secondary description or tagline */
subtitle?: string;Checklist
The comments explain that the |
- Add 'subtitle' to fields.feed array in lib/fields.js - Add subtitle property to TypeScript Output interface in index.d.ts Co-authored-by: openhands <openhands@all-hands.dev>
rbren
force-pushed
the
add-subtitle-feed-field
branch
from
fc6200d to
596ea26
Compare
|
@OpenHands add more tests for the new logic |
|
I'm on it! rbren can track my progress at all-hands.dev |
- Add subtitle parsing support for Atom feeds in buildAtomFeed() - Create test input files for RSS and Atom feeds with subtitle - Create expected output JSON files for subtitle tests - Add test cases for parsing subtitle from RSS and Atom feeds - Update existing test output files (heise, feedburner, many-links) to include subtitle field Co-authored-by: openhands <openhands@all-hands.dev>
|
🤖 OpenHands is reviewing this PR. Trigger: PR opened at This comment was posted by an AI agent (OpenHands). |
2 participants
Summary
This PR adds the
subtitlefield to the feed fields, allowing the RSS parser to extract and include subtitle information from RSS/Atom feeds.Changes
'subtitle'to thefields.feedarraysubtitle?: string;property to the TypeScriptOutput<U>interface for proper type supportTesting
All existing tests pass (38 tests). The new field integrates seamlessly with the existing field parsing logic.
@rbren can click here to continue refining the PR