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

Please make the app compatible with NC30 #776

Open
elpraga opened this issue Sep 24, 2024 · 9 comments
Open

Please make the app compatible with NC30 #776

elpraga opened this issue Sep 24, 2024 · 9 comments

Comments

@elpraga
Copy link

elpraga commented Sep 24, 2024

It would be great to see it working on the newest NC version!

@sirdrakeistda
Copy link

I have NC30 running with facerecognition enabled. facerecognition does not recognize the pdlib installation with php8.3. Not sure wether it's related to NC30 or php8.3. The pdlib test work without any error.

@tuxArg
Copy link

tuxArg commented Oct 3, 2024

I have NC30 running with facerecognition enabled. facerecognition does not recognize the pdlib installation with php8.3. Not sure wether it's related to NC30 or php8.3.

I'm running NC 29 with php8.3, and it works ok. It must be related to NC30.

@vwbusguy
Copy link

vwbusguy commented Oct 3, 2024

pdlib is working here for me with Nextcloud 30 and PHP 8.3. Clustering works for the background process. What's not currently working for me is the "Review Found people". The clusters response eventually returns (after ~30 seconds), but then it tries to call a /core/templates/message.html location that is 404 and then silently fails.

@vwbusguy
Copy link

vwbusguy commented Oct 3, 2024

It looks like progress on this is in this PR: #778

@KrystinaMir
Copy link

Please update Server version 30.0.1

@vwbusguy
Copy link

Please update Server version 30.0.1

App store compatibility is by major nextcloud version, so 30.0.1 is implied.

@matiasdelellis
Copy link
Owner

Hi everyone,
There were several interesting changes before i worry about nc30, and then I had some health problems. Nothing serious.. 😉

Here I leave you a test version. I hope to release it this week..

@elpraga
Copy link
Author

elpraga commented Nov 10, 2024

@matiasdelellis The version from the previous post seems to be working correctly on Nextcloud 30.0.2 (people found can be reviewed and so can ignored people)

Thank you for your work!

@vwbusguy
Copy link

New release is generally working for me on 30.0.2 as well

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