Skip to content
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

news:updater:all-feeds will be deprecated with news 15.0.0 #21

Open
Grotax opened this issue Sep 29, 2020 · 13 comments
Open

news:updater:all-feeds will be deprecated with news 15.0.0 #21

Grotax opened this issue Sep 29, 2020 · 13 comments

Comments

@Grotax
Copy link
Member

Grotax commented Sep 29, 2020

https://github.com/nextcloud/news/blob/master/lib/Command/Updater/AllFeeds.php#L41

The new command, that is replacing the old one is news:feed:list, behavior is a bit different as the view is per user and not a total overview.

Related PR nextcloud/news#750

@john-2000
Copy link

Hi apologies but what does this mean for my configuration ? do I need to change anything

@Grotax
Copy link
Member Author

Grotax commented Oct 5, 2020

It means that the code requires changes, I started it but found it a bit difficult to implement and lost my motivation.

The command that is used right now will be deprecated.

So the functionality has to be build in a different way, there is a nextcloud command to get all users then you would need to iterate over all users and get thier feeds and create a return value as it was before

@john-2000
Copy link

Hmm, not sure I can help.
My feed updates completely failed with the news updater but I have been able to update them with

sudo -u www-data php -f occ news:updater:update-feed "john" 43

one by one, is there a better setup I can choose - apologies I have always used the news updater and thank you for all your sterling effort on this project

@Grotax
Copy link
Member Author

Grotax commented Oct 5, 2020

That's probably an issue of news itself at the moment, I think it might be fixed in the next release.

@Grotax
Copy link
Member Author

Grotax commented Oct 5, 2020

Else you can fall back to using the system Cron in future.

@brot
Copy link

brot commented Oct 6, 2020

Yes, the python news-updater needs an update, because the news app changed the API
nextcloud/news@d00d1ab

They switched the parameter order in the "news:updater:update-feed" command. Now it's user-id + feed-id instead of feed-id+user-id

@Grotax
Copy link
Member Author

Grotax commented Oct 6, 2020

Oh I didn't see that, this would be an easy fix the other one is more complicated

@brot
Copy link

brot commented Oct 6, 2020

Is the news-updater still maintained? The last release was 2018.

The reason I ask is if I should just patch my version or if I it's possible to provide pull-requests to improve the code base. But therefore the project should be maintained, because someone needs to approve pull-requests and release new version to pypi

@DriverXX
Copy link
Contributor

DriverXX commented Oct 6, 2020

I fixed this issue in my custom version of the news-updater.
It can be found here: https://github.com/DriverXX/nextcloud-news-updater/

At the moment you need to set apilevel = v15 to make the news-updater work with the newer version of the News app.

If I understood correctly, previously, the API of the news app were versioned and the news-updater, supported API v1-2 or v2.

Thus, I wanted to ask @Grotax if this newer version of the API is the v3, or if they no longer version the API and we assume that the version of the API matches the version of the news APP.

@BernhardPosselt
Copy link
Member

Just provide a PR, then I'll publish a new version.

@Grotax
Copy link
Member Author

Grotax commented Nov 10, 2020

@DriverXX We haven't decided on that but for me it makes sense to base the api level on the major version of news

@bcutter
Copy link

bcutter commented Dec 1, 2020

OK guys today - 8 days after updating NC to v20 including several apps, including NC News - I realized there are no feed updates. Which way is the best one to fix this?

  • News app version is 15.0.6
  • nextcloud_news_updater in /usr/local/lib/python3.7/dist-packages (11.0.0)
    ...and it doesn´t update anymore... what a breaking change. 👎

I totally rely on the news-updater and now NC news is literally newsless aaahm useless :-( :-( :-(

When running on CLI I only get plenty (for every single feed) of those:

Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/nextcloud_news_updater/api/updater.py", line 34, in run
    self.update_feed(feed)
  File "/usr/local/lib/python3.7/dist-packages/nextcloud_news_updater/api/cli.py", line 80, in update_feed
    self.cli.run(command)
  File "/usr/local/lib/python3.7/dist-packages/nextcloud_news_updater/api/cli.py", line 12, in run
    return check_output(commands)
  File "/usr/lib/python3.7/subprocess.py", line 395, in check_output
    **kwargs).stdout
  File "/usr/lib/python3.7/subprocess.py", line 487, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['php', '-f', '/var/www/nextcloud/occ', 'news:updater:update-feed', '344', 'Username']' returned non-zero exit status 1.

@bcutter
Copy link

bcutter commented Dec 1, 2020

Here´s the GitHub hero who solved it. Works great.

#22 (comment)

It really is a problem this project is not actively maintained anymore - or maybe no one uses it, otherwise there would be more users wondering why their feeds are not updated. Anyway, good for the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants