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

Unified RSS Services to use 1 service #155

Closed
wants to merge 2 commits into from

Conversation

arminrosu
Copy link
Collaborator

Hello,
So instead of using n services which poll YQL for an rss, I modified the RSS Service to be compatible with any feed we currently have (and hopefully anything you might throw at it).

  • removed superfluos rss services
  • moved rss service favicons in subfolder
  • favicons are grabbed dynamically (though you can disable this)
  • added documentation for RSS "Recipes"
  • removed formspring (now sprin.me, no RSS anymore)
  • removed posterous (closed)
  • updated Facebook page example to Ars Technica (old ID was no longer valid)

Also, the author object is rather preliminary, until I have a chance to bring the templates in to the global settings.

And yes, using James Bond as a placeholder seems pretty corny in retrospect and I hope you have a better idea.

* removed superfluos rss services
* moved rss service icons in subfolder
* added documentation for RSS "Recipes"
* removed formspring (now sprin.me, no RSS anymore)
* removed posterous (closed)
* updated Facebook page example to Ars Technica (old ID was no longer valid)
@christianvuerings
Copy link
Owner

@arminrosu seems like there is a merge conflict here, could you update the PR?

@arminrosu
Copy link
Collaborator Author

yeah, because of pocket (which would be a great recipe - sorry @etano). will do soon.

@christianvuerings
Copy link
Owner

Closing PR because of merge conflicts. Definitely feel free to reopen when they're resolved.

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.

None yet

2 participants