Skip to content
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

Ensure Tenant UI is setting auto-verifier flag correctly #617

Closed
swcurran opened this issue May 27, 2023 · 3 comments
Closed

Ensure Tenant UI is setting auto-verifier flag correctly #617

swcurran opened this issue May 27, 2023 · 3 comments
Labels
TenantUI Tickets related to the Tenant UI Vue/Node app

Comments

@swcurran
Copy link
Contributor

swcurran commented May 27, 2023

When I submit a presentation request and it completes (Wallet responds), the state of the presentation request remains "presentation_received" whether or not I set the "Auto Verify" toggle to true. It would be nice if:

  • when toggled on, the "Auto Verify" caused the PR to be verified, the result displayed and the verification result displayed on the list of PRs screen.
  • when not toggled on, and the status is "presentation_received", add a button "Verify" (or whatever) to the line in the PR list, and when clicked, trigger the verification of the received presentation.

AC

  1. Ensure Tenant UI is setting auto-verifier flag correctly
  2. if it is set correctly and still does not work , log an issue in ACA-Py
@loneil loneil added the TenantUI Tickets related to the Tenant UI Vue/Node app label May 29, 2023
@loneil
Copy link
Collaborator

loneil commented May 30, 2023

Created #627 to deal with the second bullet point above. This ticket will remain to look into the Auto Verify flag.

@hiteshgh hiteshgh changed the title Verification, Presentation Request "Auto Verify" doesn't seem to do anything Ensure Tenant UI is setting auto-verifier flag correctly Jun 6, 2023
@esune
Copy link
Member

esune commented Jul 4, 2023

The state is being pulled directly from the presentation exchange in Based on aries-rfcs//0037-present-proof, the last state should be done base on the RFC. If the state is not changing, the issue might need to be reported in ACA-Py.

@esune
Copy link
Member

esune commented Nov 8, 2023

@loneil is there anything else we need to do for this, or can we close it?

@esune esune closed this as completed Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TenantUI Tickets related to the Tenant UI Vue/Node app
Projects
None yet
Development

No branches or pull requests

3 participants