search

openid / OpenYOLO-Android

Android protocol for credential exchange and update - "You Only Login Once"
http://openid.net/wg/ac/
Apache License 2.0
115 stars 18 forks source link

Feedback on demo app #144

Closed riyaz closed 6 years ago

riyaz commented 6 years ago

Well written demo app with recent best practices however when I wants to see a sample piece to integrate with my app or how it works having these extra bells and whistles is deviating from my main intention.

dxslly commented 6 years ago

Hi Riyaz, thanks for the feedback. I think you make a good point. For the moment I would recommend taking a look at the getting started guide.

Have you seen it? I would love your feedback to ensure it can help with this issue and is easily discoverable.

riyaz commented 6 years ago

Yes, library flow itself is simple so integration is easy used this too as ref https://blog.agilebits.com/2017/10/26/integrate-1password-into-your-android-apps/

Still couldn't understand hint flow, where it will be used, how it will be used?

Also, right now works well with 1password not sure how to test with Google SmartLock, I use Android O and SmartLock is enabled, but using 1password as auto fill service not sure that matters, but OpenYOLO should be independent of auto fill I suppose.

StanKocken commented 6 years ago

@riyaz the hint flow would be used during the signup process. You usually require an Email, a password, maybe a name… With the hint flow the providers will be able to give most of them.

And I agree with the sample app, we should have something well simpler to demo the usage, we will see what we can do.

StanKocken commented 6 years ago

I talked briefly with some others contributors of this repository, I will work on a very basic login/password application so it will be easier to jump on it and then close this ticket.

StanKocken commented 6 years ago

A new demo app, way simpler, has been added to the repository to address this issue.