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

Presentation Request explorer initial concept #514

Merged
merged 1 commit into from
May 10, 2024
Merged

Conversation

loneil
Copy link
Contributor

@loneil loneil commented May 8, 2024

A very simple version of the Presentation Request Explorer concept that we can iterate on given feedback from Kim and other planning.

Navigating to {vcauth url}/ver_configs/explorer loads HTML page. This is a public route.
"ver_configs" is the same router that the API endpoints are on, can adjust route naming and things later on feedback

The UI is just a Jinja template that takes in the output of fetching all Verification Configs.

  • Serves an HTML page
  • CDN for Bootstrap
  • The very basic https://datatables.net/ via CDN
  • jQuery CDN (yes in 2024, but datatables requires it and that fits the needs so far)

Can revisit this stuff as more requirements come, extend to a SPA if needed, etc. Depending on future needs if much more business logic (any authentication, sending requests for create/delete pres reqs, contacting via email, etc etc) is needed then moving to a separate standalone FE app rather than just serving an HTML page from VCAuthN API might work better.

image

image

@loneil loneil requested a review from esune May 8, 2024 23:01
@loneil loneil force-pushed the feature/presReqExplorer branch from 2becfe0 to f3f7f0b Compare May 9, 2024 16:19
@esune esune merged commit 4928c47 into main May 10, 2024
5 checks passed
@esune esune deleted the feature/presReqExplorer branch May 10, 2024 16:51
@loneil loneil mentioned this pull request May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants