bcgov / bc-wallet-mobile

BC Wallet to hold Verifiable Credentials
Apache License 2.0
62 stars 49 forks source link

Proof notification content to use from proof request name #2177

Open knguyenBC opened 2 months ago

knguyenBC commented 2 months ago

Description of feature / user story

Currently, proof request notifications reads "Proof_requested" in the Notifications tab. The content in the Notification should state who is making the request.

Acceptance Criteria

Wireframes or relevant image assets / links

loneil commented 1 month ago

@knguyenBC this might be related to what I describe here https://github.com/bcgov/bc-wallet-mobile/issues/2125 ?

Note I'm able to control the label in the notifications with 1.0 protocol proofs, and we were defaulting that to "proof_requested" in VCAuth-N but have since fixed that to be the proof config name. In 2.0 protocol proofs it's showing as blank in the wallet now.

Above is all for connectionless proofs, but there is a name field in the proof that can show up there (though appears to not work when the wallet is handling 2.0 protocol)

knguyenBC commented 1 month ago

This ticket was meant to capture (broadly) what we were talking about. Thanks for linking the tickets. I didn't realize it had already existed.