Conversation
|
@reidcooper if you are around, this PR starts to work, so tests welcome already:
What mainly lack is the finalisation of the automatic refreshing of the dynamic OPMLs (the manual button works already, including muting/disabling feeds that are not in the OPML anymore) and a number of edge cases (such as the tricky case of feeds with HTTP redirections). |
|
@Alkarex Hey hey! That's great news. Thanks for the update and powering through the update. I can give the beta a shot and see how it goes. I may be able to get to it this week. |
Good point, I have disabled adding new feeds to a dynamic category. However, it is still possible to drag & drop an existing feed to such a category. Indeed, I would like to keep the option to move feeds out to another normal category. But then, what if you change your mind and want to move it back? Suddenly we would have to keep track of which feeds would be allowed or not, which would require a bunch of custom code and would not necessarily result in a better experience.
Feeds, which URL is not in the dynamic OPML will be muted (disabled) |
|
I would suggest to use a "official" OPML logo (see also: scripting/opml.org#5): Right now there is no icon for dynamic OPML, so I created a new one: |
|
Should we add also an dyn. OPML import on the import/export screen?
|
Looks great 👍🏻 |
|
I have experimented a big with your nice OPML logos and I will clearly use them for links to our OPML views 👍🏻 But for inline use, although it works ok, it is not as readable and I think I still have a little preference to a Unicode or emoji such as 🗲
|
|
I agree with the gist about being able to tell what it looks like better, but I'm not quite sure how lightning/electricity relates to OPML. :-) |
The ideas I wanted to convey are: dynamic / magic but also dangerous |
@math-GH Ok, something like that?
|
|
Let's merge that to get some more tests and potentially come back to the cosmetic details in another PR. |
|
@Alkarex I was finally able to try out the Docker Edge release here, added an OPML feed, and it worked! I was able to see incoming feeds sync and mark them as read. Let me try this out for a few days and see how it performs. Sorry for the late reply and thank you very much for your hard work on developing this feature <3 |
| // Merge existing and import attributes | ||
| $existingAttributes = $feed_search->attributes(); | ||
| $importAttributes = $feed->attributes(); | ||
| $feed->_attributes('', array_merge_recursive($existingAttributes, $importAttributes)); |
|
|
||
| $html = getHtml($feedSourceUrl, $attributes); | ||
| $cachePath = FreshRSS_Feed::cacheFilename($feedSourceUrl, $attributes, FreshRSS_Feed::KIND_HTML_XPATH); | ||
| $html = httpGet($feedSourceUrl, $cachePath, 'html', $attributes); |
|
A little article on the Fediverse https://framapiaf.org/@freshrss/115395177812589475 |
4 participants
#fix #4191
https://freshrss.example.net/?a=opml&get=c3&token=...Background reading:
http://web.archive.org/web/20060101040915/http://www.reallysimplesyndication.com/2005/10/13
https://techcrunch.com/2005/10/16/my-thoughts-on-reading-lists/
http://scripting.com/2017/06/26.html#a080636
(Future PR) Better security tokens will be added to make the FreshRSS publication of dynamic OPML / reading lists more useful.
(Future PR) Some more documentation for this dynamic OPML feature. Help welcome.
(Future PR) The dynamic OPML export is not visible at the moment (only in the HTML code with a standard as shown below), so that should be improved somehow. Help welcome.
There are a few other remaining edge cases, such as when feeds with a URL that has been (permanently) redirected, and feeds referenced in more than one dynamic OPML.