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 RSS tile #347

Open
hvelarde opened this issue Nov 11, 2013 · 3 comments · May be fixed by #583
Open

Add RSS tile #347

hvelarde opened this issue Nov 11, 2013 · 3 comments · May be fixed by #583
Assignees
Milestone

Comments

@hvelarde
Copy link
Member

This could be based on code from the RSS portlet.

@ghost ghost assigned hvelarde Nov 11, 2013
@hvelarde hvelarde modified the milestones: 1.0a8, 1.0a7 Feb 20, 2014
@hvelarde hvelarde modified the milestones: 1.0a9, 1.0a8 May 2, 2014
@kaerumy
Copy link

kaerumy commented May 1, 2015

I need this too, and will look into it.

@hvelarde hvelarde assigned rodfersou and unassigned hvelarde Oct 2, 2015
@hvelarde
Copy link
Member Author

hvelarde commented Oct 2, 2015

@datakurre we just received a request to create a RSS tile for cover but I know there is already one in:

https://github.com/plone/plone.app.standardtiles/blob/master/plone/app/standardtiles/rss.py

and I don't want to duplicate code and efforts; can you give us a light on how we can reuse this tile, and probably others also?

if we can find a way to make tiles compatible it would benefit everybody at the end (#81).

@datakurre
Copy link
Contributor

@hvelarde @rodfersou

Well, it's the Cover which adds a lot of behavior or tiles with IPersistentCoverTile interface. I'd guess it should be made sure that Cover adapts tiles explicitly into IPersistentCoverTile before accessing those methods in Python code. Then create an adapter for that interface for the RSSTile.

@hvelarde hvelarde modified the milestone: 1.0a12 Oct 5, 2015
@rodfersou rodfersou linked a pull request Jan 20, 2016 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants