-
Notifications
You must be signed in to change notification settings - Fork 52
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
Revocation content missing from Revoked Credential Details #965
Comments
Ticket updated with wireframes and link to acceptance criteria of existing revocation ticket. |
@knguyenBC, please update acceptance criteria and wireframes |
@knguyenBC is this ticket ready to go now? |
I've updated the ticket to include the changes as discussed with the team: removed the need to invoke another notification pop up. The revocation status is permanent and contains both the revocation message and date of revocation. The notification at the top is dismissible once and cannot be brought back again. |
This is working as designed in build 1077. |
Bug:
missing (i) icon next to revocation status. This (i) icon opens the revocation messageMissing content at the bottom of the screen like revocation message and precise revocation date
Fix:
Design fixes. See new Acceptance Criteria below and wireframes
Acceptance Criteria
[from ticket #63]
Wireframes
Revocation message in credential detail screen
Screenshots of build 732
Environment
BC Wallet build 732
Tested on Android 11
Workaround
None known
Severity
Medium
Priority
Medium
Test Impact
T011.1-Proof, T011.2-Proof
The text was updated successfully, but these errors were encountered: