-
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
Suggestion: Add documentation around requiring a GameObject named SignInCanvas #1
Comments
@nitrixion hi, can I quickly ask, did you make it work in 2018.2? it seems i'm missing a step, i can't get the browser to show up on android nor ios. |
Hey! Yes, I'm currently using the code in this repo as a reference for an app in Unity 2018.2.12f1 on Android. Note: It will not work in editor or on emulator, at least I didn't have much luck getting it to work there. On android, if you export the project then debug through Android Studio, is there anything interesting in logcat or debug output? |
@nitrixion Sure i'll update the README as suggested |
+tried with @nitrixion thanks for the real device suggestion, yes no handling for editor but spent most of my time on iOS and Android emulators but could not get browser to show up. Tried several 2018.2.* versions last one was 10f1 @peterhorsley will try logcat but unfortunately could not get a single build to pop a browser on many different macs, maybe i'm missing a step on all of them, unlikely. i was about try downgrading to exact version you used to confirm. but now i will try real device first and report. |
documentation added to readme for gameobject named signincanvas, closing this issue as per original report. |
First off, thanks for this example implementation. It has been extremely helpful in getting up and running with OAuth through IdentityServer in Unity.
Currently, the Unity Implementation expects a GameObject named "SignInCanvas" to have a script with an "OnAuthReply(string)" method to receive the response from the platform specific web view. I have not found this documented anywhere in this repo.
My suggestion is to add a line to the README documenting this. Additionally, a config option to change this name would be useful.
Thanks again for this extremely useful example!
The text was updated successfully, but these errors were encountered: