Skip to content

Conversation

@infogulch
Copy link
Contributor

@infogulch infogulch commented Feb 12, 2025

Call Feed.OriginalFeed() to get the original feed if available.

Fixes #229

I just threw this together real quick to see what you think about it.

Call Feed.OriginalFeed() to get the original feed if available.
@infogulch
Copy link
Contributor Author

infogulch commented Feb 12, 2025

Ah of course the test expected values don't match anymore. Not sure if the expected values should be updated or if the tests should ignore the originalFeed field since it's not exposed anyway.

@infogulch
Copy link
Contributor Author

A different implementation could change the definition of Translate() to return three values: the general Feed and the specific feed object along with the error. Then there would be no need to add a field that affects the test result output. Though this strategy is not very backwards compatible to say the least.

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.

RSS "source" attribute

1 participant