Closed crazy-max closed 6 years ago
Hi @crazy-max !
This is a picofeed issue, can you open this issue there?
Hi @BernhardPosselt !
Ok i will add this issue there.
Indeed it's a picoFeed issue (example in Parser/Atom.php
file) :
$item->setDate(max($published, $updated)); // We use the most recent date between published and updated
I have created an issue on their repository.
For the moment i have added a method to retrieve the published version on picoFeed parsers and change the code in the feedfetcher.php
script of ownCloud News :
Maybe you could create a pull request for picofeed if you've already fixed it ;)
Keep in mind that not all feeds feature the published tag so a fallback should be considered
Yep i have a fallback too i will create a pull request ;)
I have made a comment on the picoFeed issue with the implementation of published / updated item date : https://github.com/fguillot/picoFeed/issues/267 I think i will fork ownCloud News too and add an implementation to let the choice to the user to choose between updated or published date.
If we have a way to distinguish between updated and published, I think it would make sense to have both. Let's first get the picofeed PR merged, then start working on the News stuff ;D
Some tips:
That and maybe some unit test adjustements should be all for the server side.
These are the places where the client side date is being rendered:
We'd have to think of a way to integrate this in a way that is less intrusive:
Totally agree but like you said let's focus on picoFeed first ;)
Hi,
First of all thanks for this great app! Yesterday i have migrated from Feedly to ownCloud News app. Eveything seems fine except a "problem" with the items timestamp.
Feedly seems to be based on the
<published>
feed tag and ownCloud News is based on the<updated>
feed tag to create the item timestamp.For example on a Youtube RSS feeds channels the updated timestamp is over and over updated so the timestamp is not accurate for me (see screenshots and this feed)
Is there an option to only display published date and not updated date in the column ?
I saw a bugfix on 4.2.0 release, maybe it is related ? :
ownCloud : 9.0.1 ownCloud News: 8.7.1
Thanks!