-
Notifications
You must be signed in to change notification settings - Fork 22
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
No longer works with tinytinyrss #144
Comments
Same issue here. tt-rss seems to handle plugins a bit differently since last commits, the route for bookmarking changed to Likely introduced by this commit https://git.tt-rss.org/fox/tt-rss/commit/273ada7353b185e20452d54a8206d5e0cef9e573 |
@TheCoffeMaker Looks like Mozilla will soon be removing page actions from the URL/awesome bar, and that'll be the end of Awesome RSS. First the removed the RSS when the removed RSS from Firefox, now they're removing the "Awesome" part with the removal of page actions. |
@shgysk8zer0 Sorry for the off-topic question, but any more info on that? |
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/pageAction does not mention any forthcoming deprecation, any news on this ? |
I am also affected. Would it be possible to make this change? I understand Firefox might one day change, but I see this was feared since at least 2018 (see #92). Letting user set the URL completely with parameters just like https://example.org/homegrownrss.php?foo=bar&url=%s would make sure you don't have to update for every change in nextcloud, tt-rss or every obscure service there is. |
Why wasn't this updated sooner? It was literally one word, took me like 5min total. Well it's out for PR now, confirmed working locally for me! This never effected me until recently because I had never updated my TT-RSS instance. |
The problem is still there. Such a shame. @hockeymikey says he has a fix, if I understand correctly. Any chance to merge #165 and release ? |
Here you go: https://cloud.hockeymikey.com/s/ie76xf3sjx7r6fg |
Is there a reason why the Pull Request #165 isn't merged? |
This is affecting me also. I took a look at PR #165 and it is very trivial and easy to identify as not malicious (Changing only 'subscribe' to 'bookmarklets--subscribe'). I hope you can find a few minutes to do another release with this merged in. Thanks! |
As a work around: Go into tt-rss preferences, in the 'feeds' tab, under Bookmarklets. Drag the 'Share with Tiny Tiny RSS' button into your toolbar or bookmarks. Then you can just click it to sub to tt-rss on any page without awesome rss extension. |
Many thanks for pointing this! 👍 I had to enable Bookmarklets in the Plugin section (and click on Enable Selected). The "Subscribe in Tiny Tiny RSS " button appears in "Feeds"/"Plugins". It seems to work. |
Describe the bug
TinyTinyRSS API changed and not returns the following error when awesome-rss tries to add a new feed:
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Just expect to get the feed added correctly to tinytinyrss.
Screenshots
N/A
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: