identity-samples icon indicating copy to clipboard operation
identity-samples copied to clipboard

Compose sample for CredentialManager

Open yschimke opened this issue 1 year ago • 16 comments
trafficstars

Adapting the sample for compose instead of fragments seems complex. I've done this before, but also made mistakes, so a compose sample would help the adoption.

I guess Compose + MAD, such as ViewModels, which complicate where you hoist auth logic and state.

yschimke avatar Apr 24 '24 07:04 yschimke

Found https://github.com/Dashlane/android-passkey-example

Which I have no affiliation with, but is quite helpful.

yschimke avatar Apr 26 '24 20:04 yschimke

Hi We are working on this, you will get this by next month. The code is in review phase

There's a rough branch which has atleast the Compose and MAD (we are working on a refined MAD practices and clean version), please check glitch_me_sample branch

niharika2810 avatar Apr 28 '24 19:04 niharika2810

Thanks, great news.

Do you want any feedback on it?

yschimke avatar Apr 28 '24 21:04 yschimke

Hey, are you asking about current branch or the one which we are refining. Once we upload that code on main branch (somewhere in May), please feel free to share the feedback :)

niharika2810 avatar Apr 29 '24 05:04 niharika2810

How is this coming along?

ber4444 avatar Jul 22 '24 20:07 ber4444

Hey this is in review, will be out once done. Thanks!

niharika2810 avatar Jul 29 '24 04:07 niharika2810

Hi We are working on this, you will get this by next month. The code is in review phase

There's a rough branch which has atleast the Compose and MAD (we are working on a refined MAD practices and clean version), please check glitch_me_sample branch

Hello Niharka, I see that the branch you mention was last updated 1 year ago you yesterday's comment says "this is in review" - did you mean glitch_me_sample branch with Compose ?

aarchish avatar Jul 30 '24 14:07 aarchish

I believe this means an internal Google review process, that all samples need to go through before they are published to GitHub.

yschimke avatar Jul 30 '24 17:07 yschimke

I believe this means an internal Google review process, that all samples need to go through before they are published to GitHub.

Hello yschimke, I was specifically asking this in regard to previous context of using Compose and MAD with Credential Manager API

aarchish avatar Jul 31 '24 12:07 aarchish

Hello @niharika2810,

are we still waiting for the PR to go through?

aarchish avatar Aug 06 '24 11:08 aarchish

Hey we are still refining a few bits. It should be there this week. If you want to see on a high level, you can check the compose branch as of now. Branch name is https://github.com/android/identity-samples/tree/credman-compose/Shrine

We will keep you posted. Internal reviews take some time. THank you all the patience.

niharika2810 avatar Sep 01 '24 20:09 niharika2810

@niharika2810 I tried the Compose sample, it was very simple to use, worked first time. Nice work.

yschimke avatar Sep 23 '24 09:09 yschimke

Great 😃 Thank you Yuri

On Mon, Sep 23, 2024, 3:26 PM Yuri Schimke @.***> wrote:

@niharika2810 https://github.com/niharika2810 I tried the Compose sample, it was very simple to use, worked first time. Nice work.

— Reply to this email directly, view it on GitHub https://github.com/android/identity-samples/issues/76#issuecomment-2367741978, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJXAH436NHTVO47MGUY6FETZX7QVHAVCNFSM6AAAAABGWMEML6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRXG42DCOJXHA . You are receiving this because you were mentioned.Message ID: @.***>

niharika2810 avatar Sep 24 '24 06:09 niharika2810

Can the sample be merged into the main branch? It's quite hidden.

yschimke avatar Feb 07 '25 17:02 yschimke

@yschimke Agreed, I was just looking for this!

JohnUgwuadi avatar Feb 07 '25 18:02 JohnUgwuadi

Yeah right, there are a few features not being ready for Compose. We are waiting on that. This should be merged soon.

@cy245 FYI

niharika2810 avatar Feb 07 '25 21:02 niharika2810