-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Issues with the Nextcloud SSO and Android 11? #4
Comments
Hi @RAYs3T Thanks.. 😉 |
OK this sounds like our problem. Will confirm it tomorrow with a fresh build! |
Hmm interesting, this still does not work (tried with current master) |
Hi @RAYs3T https://delellis.com.ar/s/f4mZ3YpoWZtm3dC See nextcloud/Android-SingleSignOn#282 for more details, but need your feedbak about that.. 😄 |
Another apk.. Test this.. 🙈 |
Hi @Thovi98 @RAYs3T I need to try 3 releases. And on the other hand, a new version of the SSO library. First with sdk 29 and then with 30, and please confirm if work the APK in the respective PR EDIT: Update link. |
@matiasdelellis I'll test these today. You Nextcloud link includes an empty file and I don't have write permissions to it (can only download it) |
Thanks @RAYs3T ... Fix that.. |
I've testet both APKs from the PRs one of them (20201119124830 - quicknotes-query-master-release.apk) works! - WIll now test the master branc |
@matiasdelellis Master works as well with a fresh build! Awesome! |
After same tests, I come to same conclusion ! So I don’t think it’s necessary to confirm in the respective PR 😄 |
Hi, There was no change in master since you said it didn't work here. 😕 🤔 |
I'll delete all userdata and cache and test it again. Maybe something is persistent over installs which affects this. |
It works on master now... @Thovi98 and if you use the current master it works for you as well? So somehow installing the app from the PR once influenced this result? I still think the changes from the PR are required @matiasdelellis, but i can't explaint why this works still after deleting cache, storage and uninstalling... |
I’ve just tried again with this https://delellis.com.ar/s/ei58ALDyrGHgDeY (is it the right to test ?) and worked perfectly ! |
Hey! Is it possible to get the last working APK? Thank you 😃 |
Please, I use a lot this app on NextCloud as it's being my replacement for Google keep, can I get the last APK with SSO fixed? Want to actively contribute to the project 😊 |
Can I snag an apk to test (the last link above is expired Nov 2020)? I also have an android 11 device. |
Pulled from the git and quickly made own app.. errors out "cant determine server side quick notes app." Sigh. |
Hey @Joshndroid ! Hope it will let you use the amazing QuickNotes app on your Android 🎉 |
Hey mate. Thanks for the upload I appreciate it. I am on Android 11 on oxygen OS. I installed the app fine, went through picking server then clicked allow button on the pop up. I can see the blank initial home page. So, so far so good. Thanks again Edit - initial note creation is working from app and can see it sync'd in the online quicknotes in nextcloud on Firefox |
To whomever it may interest, I found a version of a working Quick Notes Android Client fork here (Release Oct 2023): |
Could it be that the nextcloud SSO is somehow not working on android 11?
I did dig around a little bit and found out that indeed the nextcloud-android-sso is returning an error for
AccountImporter.pickNewAccount(this)
com.nextcloud.android.sso.exceptions.NextcloudFilesAppNotInstalledException
I have the official live build of nextcloud client installed, running on a Pixel 4 XL with latest Android 11.
What android are you testing on?
If someone could reproduce this I would raise an issue over at https://github.com/nextcloud/Android-SingleSignOn
The text was updated successfully, but these errors were encountered: