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

Select which Ledger/Endorser to connect to - Tenant UI #681

Closed
2 tasks
esune opened this issue Jul 4, 2023 · 5 comments · Fixed by #790
Closed
2 tasks

Select which Ledger/Endorser to connect to - Tenant UI #681

esune opened this issue Jul 4, 2023 · 5 comments · Fixed by #790
Assignees
Labels
TenantUI Tickets related to the Tenant UI Vue/Node app

Comments

@esune
Copy link
Member

esune commented Jul 4, 2023

As a tenant, I should be able to pick which ledger my agent should have write privileges on - when running in a multi-ledger instance.

Logic flow:

  • New tenant request
  • Innkeeper authorizes tenant to become issuer
  • Tenant selects ledger/endorser to be rooted on from list of available options
  • Once rooted, no change is allowed

Acceptance Criteria:

  • Invoke innkeeper APIs to select ledger/endorser
  • Update UI/UX to provide tenant with appropriate selection controls - innkeeper would just authorize the tenant to become an issuer, the tenant selects which ledger to be rooted on
@shaangill025 shaangill025 self-assigned this Jul 12, 2023
@loneil loneil added the TenantUI Tickets related to the Tenant UI Vue/Node app label Jul 18, 2023
@loneil
Copy link
Collaborator

loneil commented Jul 18, 2023

Need to possibly consider all of the decisions coming out of bcgov/DITP#31 for what the interface of this ticket would be.

cc @esune

@loneil
Copy link
Collaborator

loneil commented Aug 10, 2023

Does completion of #546 mean the ACA-Py and/or plugin API calls support what's needed for this?

For grooming/planning @esune / @shaangill025 did you want me to take Tenant UI frontend part of this for next sprint then?

@esune
Copy link
Member Author

esune commented Aug 10, 2023

@loneil correct, #546 exposes the APIs required for this to work. @shaangill025 please coordinate with @loneil for the UI/UX design and development.

@hiteshgh
Copy link

hiteshgh commented Sep 5, 2023

ready for testing @esune and @loneil

@hiteshgh
Copy link

hiteshgh commented Sep 6, 2023

@shaangill025 to log bug

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

Successfully merging a pull request may close this issue.

4 participants