Re: syndication and rendering of content

2014/1/9 Nathan Van Gheem <[email protected]>

> Hi Juan,
>
> Thanks for the help in implementing this!
>
> This looks familiar to how franco implemented newsml and I had told him it
> was the incorrect way to do so.
>
> You should not need to have an newsml specific folder and collection
> adapters. It also looks like you have duplicated code. Anything custom to
> how newsml is rendered should go into NewsMLFeedView
>

Agreed, I was just porting code, but you're right we could simplify it...
maybe was ok and working for a collective product, should be improved to
get into the core... I can do it...


>
> I do not think the newsml_enabled_types option is necessary either.
>

Agreed,


>
> As for rendering the content body, didn't we also want title and
> description in there? Maybe we should render the content-main macro?
>

not here... title and description are part of the metadata in every feed
entry... not need to duplicate it into the body... and I think than most
feed readers didn't expect title and desc. in there...

>

best regards,
-- 
Juan Pablo Giménez
skype & twitter: jpggimenez
Simples Consultoria <http://simplesconsultoria.com.br/>

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk

_______________________________________________
Plone-developers mailing list
Plone-developers-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/plone-developers