collective / collective.nitf

A Dexterity-based content type inspired on the News Industry Text Format specification
8 stars 3 forks source link

byline field must be honored in syndication #93

Closed hvelarde closed 9 years ago

hvelarde commented 10 years ago

by default, Plone uses standard author field on syndication feeds and that means a news article will display the name of the person who created it and not the one who actually wrote it.

we need to find out a solution (probably an adapter) involving standard syndication in Plone 4.3.

rodfersou commented 9 years ago

@hvelarde I did the adapter for Plone 4.3 here https://github.com/collective/collective.nitf/commit/1776f4a628d0397efb8c227ae9a2f6a572ee98cc but for Plone 4.2 it is not working due this line: https://github.com/collective/collective.nitf/blob/master/setup.py#L73

rodfersou commented 9 years ago

@hvelarde do you think there are any problems to change the Plone 4.3 dependency?

rodfersou commented 9 years ago

@hvelarde also 'plone.app.querystring >=1.2.5', and 'collective.js.jqueryui', version are yelling me when I try to run buildout

hvelarde commented 9 years ago

@rodfersou master branch of this package will not run in Plone 4.2; check the 1.x branch

rodfersou commented 9 years ago

@hvelarde thanks

rodfersou commented 9 years ago

Fixed in these pull requests https://github.com/collective/collective.nitf/pull/120 https://github.com/collective/collective.nitf/pull/122