-
Notifications
You must be signed in to change notification settings - Fork 20
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
Nextcloud 22 #305
Comments
Can you confirm the current version runs smoothly on nextcloud 21 ? |
I am not updating as too many apps are labelled unsupported. Hopefully
others can test. Apologies, I simply do not have time to break my instance.
Flowupload rocks btw.
…On Fri, Feb 26, 2021, 3:14 PM Jonathan Treffler ***@***.***> wrote:
Can you confirm the current version runs smoothly on nextcloud 21 ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANUKZUN6BNHZ3SNGZDQKDTTBATMJANCNFSM4YJFN6SQ>
.
|
I have upgraded and can confirm that, at least for me, it does not work with Nextcloud 21. |
Thanks for testing :) |
If i have time in the next days, i will try to find out what causes the compatibility issues on Nextcloud 21. |
@chandlben What kind of errors do you get? I tried it on Nextcloud 21 on a test instance and it works so far. |
Flow upload simply says "Error" to each uploading file but in the system logs this is generated:
I removed and then added the app again to no avail. |
Receiving the same error as well. |
I added a potential fix (bb6c554), can you test it by manually adding the changed lines and see if it works? The filesystem calls need to be set up separately now it seems. @JonathanTreffler I tried to build it again, but it seems like there is a problem with the Vue frontend code:
|
The frontend problem is likely because of a dependency update. I am currently sick, but i will look into that in a few days. |
Any news about the update? App is incompatible with NC 21 for me too. |
Sorry to add my complain, it failed with me too:
|
the same problem. |
any new update for the issue ? |
It is the only app not updated on my instance, stopping me to finally save my precious cpu (nextcloud 21 reduce load by 80% <-> 90%). |
|
The last commit was 4 days ago. Be patient.
…On Fri, Jul 9, 2021, 6:22 AM LostInCompilation ***@***.***> wrote:
Well, I think the app is dead...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANUKZUU5ZZPGTO5GZTUQMLTW3ZY7ANCNFSM4YJFN6SQ>
.
|
Nope, I just looked into the issue and the fix for Nextcloud 22 is literally adding a "/" to an input. |
Sorry for taking so long, but the frontend build is now fixed. It was a dependency update of Vue-loader, which does not support Vue version 2 anymore. At some point we will have to upgrade to vue 3, but the Nextcloud eco system sadly is not ready for vue 3 yet. |
For all of you, that have had to wait far too long: In my testing everything works, but if you have any problems with it, just comment here. |
@e-alfred I think we can publish a new release soon. |
@JonathanTreffler Thanks for your work, I will publish a new version soon if no other problems come up. |
will this also work on 21? |
Try installing it on 21 and let us know.
…On Sun, Jul 11, 2021, 10:12 AM elasticSteel ***@***.***> wrote:
For all of you, that have had to wait far too long:
Heres a "beta" release for Nextcloud 22:
will this also work on 21?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#305 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANUKZSDZ4IL5QYK44GW7BLTXHGG7ANCNFSM4YJFN6SQ>
.
|
I am sorry, I can't test it myself at the moment. I do not have a test system available at the moment :( |
Did a quick test on a fresh 21 install. It seems to work just fine :-) |
Still is happening in NC 21 |
Will there be an official release for NC21 too? Or just NC22? |
I hope and plan so, but I need to be able to reproduce the error :/ |
I've testing on NC 21.0.3 with PHP 7.4.3 and it works fine. |
for nextcloud 21 it works :) |
@JonathanTreffler I am currently building a new release, but I am getting a lot of errors at the 'npm run build' step:
|
I will look into it. Does the result of the compile work ? |
Yaay, thanks :) |
@JonathanTreffler The build works now, but I still get an error when loading the app:
|
Weird. I will try to replicate the issue. |
Hi, |
On NC21 I installed the 1.1.3 version of flowupload but I got a blank page below the NC top menu. Then I tried the zip from this thread.
It worked a few times but now it says "Completed" when the upload is done and yet I cannot find the files. I tried uploading to several folders but they remain empty even after a "completed" upload. |
NC 22.2.5 canal stable, Does not work.
|
I can verify this works in NC 22.2.6, I had to go into the "info.xml" and change the dependency version and wala! thanks guys, |
The "beta" version also works in NC23.0.3, whereas v1.1.3 does not, at least for me. |
Can confirm, 1.1.2 does work, and 1.1.3 not. Newest version just gives en error at any file. |
Same Old Situation for NC24.0.0. "Beta" works fine, v1.1.3 doesn't. |
it doesn't work. you need add "" in the function call \OC_Util::setupFS(); |
Please update for latest Nextcloud 21 release.
The text was updated successfully, but these errors were encountered: