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

Upgrade to Nextcloud 29 #114

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

redblom
Copy link

@redblom redblom commented Sep 3, 2024

Switch to vue frontend.
Fixes NC 29 compatibility issues.

Fixes #113, #110, #108

@redblom redblom force-pushed the fix-file-context-menu branch from 858d0ff to 45d83dc Compare September 3, 2024 09:01
@redblom
Copy link
Author

redblom commented Sep 5, 2024

@PaulLereverend requesting review.

@xeruf
Copy link

xeruf commented Oct 3, 2024

looks like he is gone, maybe you can publish a fork? :)

@JurgenWauters
Copy link

JurgenWauters commented Nov 4, 2024

Hi, The extract part is working OK. I only get this message in the Logging

image

{"reqId":"F20lQv5g9K338PfmPUlB","level":3,"time":"2024-11-04T10:47:30+00:00","remoteAddr":"172.17.0.1","user":"admin","app":"jsresourceloader","method":"GET","url":"/apps/files/","message":"Could not find resource extract/js/extraction.js to load","userAgent":"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/130.0.0.0 Safari/537.36","version":"29.0.7.1","data":{"app":"jsresourceloader"},"id":"6728a657f28ef"}

@JurgenWauters
Copy link

@redblom Hi Thanks for the update, Do you have time to take a look at the issue I posted above?

@redblom
Copy link
Author

redblom commented Nov 25, 2024

@redblom Hi Thanks for the update, Do you have time to take a look at the issue I posted above?

I do not experience this (cannot reproduce). Also this version of the extract app has no reference to extraction.js.
Maybe you'll find this informative: https://help.nextcloud.com/t/jsresourceloader-error-in-files/198659

@JurgenWauters
Copy link

JurgenWauters commented Nov 27, 2024 via email

@redblom
Copy link
Author

redblom commented Nov 27, 2024

@JurgenWauters oops :) I was working on an already updated in house branch.
Thanks for noticing!
Pushed the fix.

@JurgenWauters
Copy link

@redblom Big thanks for the update, is it also possible to make a new release?

@bcutter
Copy link

bcutter commented Dec 29, 2024

Yeah, where's the new release so people can actually benefit from this?

@mjeshurun
Copy link

Can someone please explain how to install this version of the app in Nextcloud 30 AIO docker?

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

Successfully merging this pull request may close these issues.

Please, make app compatible with NC29
5 participants