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

Sidebar layout broken in NC 28+ #2585

Closed
3 tasks done
msbt opened this issue Feb 10, 2024 · 30 comments
Closed
3 tasks done

Sidebar layout broken in NC 28+ #2585

msbt opened this issue Feb 10, 2024 · 30 comments
Labels

Comments

@msbt
Copy link

msbt commented Feb 10, 2024

IMPORTANT

Read and tick the following checkbox after you have created the issue or place an x inside the brackets ;)

  • I have read the CONTRIBUTING.md and followed the provided tips
  • I accept that the issue will be closed without comment if I do not check here
  • I accept that the issue will be closed without comment if I do not fill out all items in the issue template.

Explain the Problem

The feed navigation on the left side is broken, items are stacked above each other or look otherwise glitchy:

image

Steps to Reproduce

  1. Upgrade from NC 27 to NC 28
  2. Navigate to News-app
  3. ??
  4. Report bug

System Information

  • News app version: 24.0.0
  • Nextcloud version: 28.0.2
  • Cron type: system
  • PHP version: 8.1.2
  • Database and version: PostgreSQL 14.9
  • Browser and version: Firefox 122.0.1
  • OS and version: Win 10 Pro
@msbt msbt added the bug label Feb 10, 2024
@Grotax
Copy link
Member

Grotax commented Feb 10, 2024

Duplicate of #2494

News 24.x does not support Nextcloud 28, the app also does not claim to support Nextcloud 28.
On the app store you will see that the latest supported version of Nextcloud is 28, but the app store includes all per-releases for that information.

You probably force-enabled News at some point.

If you want you can upgrade to the latest alpha release of News 25.0.0-alphaX. That version includes a major re-write of the UI. Unfortunately that version is not yet feature complete see #2503. There is no roadmap and no ETA. News is not supported by the Nextcloud company but a comunity project.

To upgrade to News 25.0.0-alphaX you can choose one of the options below.

  1. Use command line occ to install the alpha
    1.1 sudo -u www-data php ./occ app:update news --allow-unstable (adjust command according to your setup)
  2. Set the update channel in the Admin settings for Nextcloud to beta and wait until the update for News is offered in the webinterface
  3. Do it manually, download release, replace files, trigger upgrade via occ

@Grotax Grotax closed this as not planned Won't fix, can't repro, duplicate, stale Feb 10, 2024
@Grotax Grotax pinned this issue Feb 10, 2024
@msbt
Copy link
Author

msbt commented Feb 10, 2024

Ah my bad, thanks for the explanation! I checked the last issues and couldn't find one that fit the description (apparently because they were closed already), but now I see there are already multiple ones, should have looked deeper. I rarely login as admin, so I didn't force-enable this time around, so not sure what happened there. Either way, thanks for your efforts and work!

@Grotax
Copy link
Member

Grotax commented Feb 10, 2024

If I'm right nextcloud remembers if you force enable an app, it writes the app name to a list in the config.php file. So it could be from long ago.

@exussum12
Copy link

@Grotax Any chance those options can be added somewhere in the readme or pinned issue? That is the best upgrade path I have seen. Thanks for that!

@Grotax
Copy link
Member

Grotax commented Feb 12, 2024

This issue is already pinned 😊

@bodograumann
Copy link

I installed the alpha version, but still get a 404 error in the android app. Is this a missing feature in the alpha version or should I open an issue over at the android app repo?

@SMillerDev
Copy link
Contributor

I don't think anyone had any issue with a 404. So that sounds like something that would be good to report to the android app for sure

@bodograumann
Copy link

Ugh, it was my own stupidity. After force-installing the app as unsupported version, I also had to activate it explicitely again. 🤦
Now it works fine afaict :-)

@kimcdow
Copy link

kimcdow commented May 2, 2024

Sidebar broken again in Nextcloud Hub 7 (28.0.4)

@FadeFx
Copy link

FadeFx commented May 18, 2024

Using "sudo -u www-data php ./occ app:update news --allow-unstable" to update, will we have to do this frequently to stay up to date, or will it auto update the alpha once it is installed?

@LMRW
Copy link

LMRW commented May 31, 2024

If someone mostly uses a 3rd party client app (I use Lire) is it safe to use nextcloud news within nextcloud 28? As I won't need the web UI, I just want the client apps I use to continue to be served

Thank you too for nextcloud news

It's one of the most interesting apps

I really appreciate it

@Grotax
Copy link
Member

Grotax commented Jun 1, 2024

Yes you can update to the latest alpha.

@LMRW
Copy link

LMRW commented Jun 2, 2024

Thank you I upgraded to the alpha, it's so much faster now, it's visibly so quick! Great results so far. Thank you.

@scubamuc
Copy link

scubamuc commented Jun 18, 2024

@Grotax,

Yes you can update to the latest alpha.

thanks for the info...

"Nextcloud News" is a great app and definitely in my personal favs list. News works fine in Nextcloud snap with some minor caveats in the interface and perfectly with the Android client.

sudo nextcloud.occ app:update --allow-unstable news enables upgrading to alpha. Can you explain how we'd be getting back to "stable" once released... or is that automatic? see nextcloud-snap/nextcloud-snap#2758 (comment)

thanks for a great app

@Grotax
Copy link
Member

Grotax commented Jun 18, 2024

Once we release a stable version you will be able to upgrade to that and nextcloud will notify you.

@silopolis
Copy link

Update to unstable fixed layout on my NC 29 :)

@scubamuc
Copy link

@Grotax, working like a charm in Nextcloud snap vs. 29.02.1
grafik

there is an issue with autopdating feeds, somehow cron is ignored although cron is active:

grafik

so it requires manual refresh to update the feeds when using the browser interface. the Android client is updating fine though.

@Grotax
Copy link
Member

Grotax commented Jun 20, 2024

You mean that the feed items in the browser are not updated?

But the android app is getting updates.
That means cron is working.

Not sure what the issue of the web app is.

@scubamuc
Copy link

@Grotax

You mean that the feed items in the browser are not updated?

yes exactly...

@scubamuc
Copy link

@Grotax some logs showing cron error;

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:43+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:43+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:43+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Rdf                                                         2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:57+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:34:58+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:34:58+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:34:58+00:00

  Debug    news               new parser added : FeedIo\Standard\Rdf                                                         2024-06-21T05:35:47+00:00

  Debug    news               new parser added : FeedIo\Standard\Rss                                                         2024-06-21T05:35:47+00:00

  Debug    news               new parser added : FeedIo\Standard\Atom                                                        2024-06-21T05:35:47+00:00

  Debug    news               new parser added : FeedIo\Standard\Json                                                        2024-06-21T05:35:47+00:00

  Debug    cron               CLI cron call has selected job OCA\Files_Sharing\ExpireSharesJob (id: 11, arguments: null)     2024-06-21T05:35:47+00:00

  Debug    no app in context  Exception: dirty table reads: SELECT * FROM `*PREFIX*jobs` WHERE (`reserved_at` <= :dcValue1)  2024-06-21T05:35:47+00:00
                              AND (`last_checked` <= :dcValue2) ORDER BY `last_checked` ASC LIMIT 1 at
                              lib/private/DB/Connection.php line 316

                              0. .../doctrine/dbal/src/Query/QueryBuilder.php line 344
                                 OC\DB\Connection->executeQuery(

                                 )
                              1. .../src/Query/QueryBuilder.php line 384
                                 Doctrine\DBAL\Query\QueryBuilder->executeQuery(

                                 )
                              2. .../DB/QueryBuilder/QueryBuilder.php line 280
                                 Doctrine\DBAL\Query\QueryBuilder->execute(

                                 )
                              3. .../DB/QueryBuilder/QueryBuilder.php line 293
                                 OC\DB\QueryBuilder\QueryBuilder->execute(

                                 )
                              4. .../BackgroundJob/JobList.php line 230
                                 OC\DB\QueryBuilder\QueryBuilder->executeQuery(

                                 )
                              5. cron.php line 163
                                 OC\BackgroundJob\JobList->getNext(

                                 )

after this cron goes away.

@Grotax
Copy link
Member

Grotax commented Jun 21, 2024

Ah ok so cron does actually stop working and then no cron job is running anymore. Therefore the News web and all clients don't get updates.

That's the opposite of what I understood before.

See #2693

@scubamuc
Copy link

scubamuc commented Jun 21, 2024

@Grotax looks like this may solve the issue:

cron not working after upgrade to 29.02.1

new version of fulltextsearch available vs. 29.0.0

grafik

https://help.nextcloud.com/t/frequent-nextcloud-29-hub-8-update-issues/189897/14

this seems to solve cron issue

@scubamuc
Copy link

@Grotax, thanks for your patience... I've had some time to sort things out.

Ah ok so cron does actually stop working and then no cron job is running anymore. Therefore the News web and all clients don't get updates.

yes, cron had stopped working (my watcher script obviously sucks) cron not working after upgrade to 29.02.1 #2793 but I got that sorted by disabling and removing FTS.

this issue with autoupdating the feeds in the browser (FF, EDGE, CHROMIUM) still remains though

there is an issue with autopdating feeds, ....

strangely the android news app works fine and shows updated feeds icon immediately. yet the feeds aren't visible in the browser sidebar until the site is refreshed manually:

grafik

now I've been watching this thread and see no mention of other folks having the same issue. is it just me or can you confirm this?

@SMillerDev
Copy link
Contributor

I think the old UI never auto updated either, that's probably why nobody mentioned it as a bug.

@scubamuc
Copy link

scubamuc commented Jun 25, 2024

@SMillerDev, thanks for you reply.

I think the old UI never auto updated either

I can't confirm that... it always updated for me. I usually have the news feeds app open in a browser tab waiting for github RSS issue feeds...

see no mention of other folks having the same issue

... maybe it is only me 👀

@Grotax
Copy link
Member

Grotax commented Jun 25, 2024

There is already an item for that in #2503

Please everyone check there first if something is not working as expected, probably it was already mentioned.

@dallyger
Copy link
Contributor

The workaround of using the cli (php ./occ app:update news --allow-unstable) no longer works for me. Says it's already up-to-date. Worked fine for me for months and now I'm stuck back at v24 with a broken interface. Upgraded from NC 29.0.6.1 to 29.0.8.1.

Is there a new workaround? Or even better, a stable 25 release? I see no real benefit of keeping the 24 version as the 'stable' one as NC 27 is EoL and nobody should be using it anymore.
Using the 25 release as the new stable would help all users of this plugin, as they no longer have to apply the workaround each time they update NC.

@bodograumann
Copy link

I had the same experience. @dallyger Though I am still on -alpha4, so my first guess was that maybe the ordering put -alpha10 and -alpha11 between -alpha1 and -alpha2.

I also tried to manually replace the files with the release from github, but then my nextcloud completely broke.
Might be because News now requires PHP 8.2 and I only have 8.1.2 :-(

@Grotax
Copy link
Member

Grotax commented Oct 23, 2024

Hello,

so two things I decided to stick to alphas until I feel like new is again on a level that the features really needed are back.

And yes we require php 8.2 now so if you are still on php 8.1 it's time to upgrade Nextcloud 30 also deprecated php 8.1 so they will drop support soon as well.
On Debian/Ubuntu which is guess most people use it is quite easy to switch to a different php version.
There is the famous ppa https://launchpad.net/~ondrej/+archive/ubuntu/php by Ondrej the php maintainer for Debian since 2000.
I usually use it in my setups and I'm happy with that.

And of course you could also check to update to the next release of your distro both ubuntu and debian moved on already.

And just to state the obvious a stable release won't change your php situation.

@dallyger
Copy link
Contributor

okay, the cli does work again. 2 days ago, it said v24 is the latest, not it updated to 25.0.0-alpha12. So it seems like it was an issue with the nextcloud app store.

@Grotax Grotax unpinned this issue Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests