feat: Multi ecosystem lookup in Trinsic Connect (#3011) * Imple #159
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lucasamonrc
approved these changes
Nov 16, 2023
|
@tmarkovski the checks are still failing after merging main. Am I doing something wrong here? |
auto-merge was automatically disabled
November 30, 2023 19:45
Pull request was closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
feat: Multi ecosystem lookup in Trinsic Connect (#3011) * Implement first multi-ecosystem lookup * Properly handle the results of initiate verification * Add Todo comment * Fix proto changes * feat: add support for multiple issuers in the cross-ecosystem bet (#3015) * feat: add support for multiple issuers in the ecosystem (except issuance on final verification) * feat: add export of simple format to API response towards connect frontend (#3018) * feat: add export of simple format to API response towards connect frontend * Simplify json parsing + mapping * Simplify credential subject parsing * Le Format * Move government normalization to shared folder as Server/ConnecTService needs it * Include Normalized Government ID in SDK output (#3028) * Add verification metadata to return value for last verification info * feat: add default setups for platform required ecosystems (#3027) * Sample starting point if we want to configure these ecosystems in code * feat: add default setups for platform required ecosystems * It's an interface * Le Options * Whoopsie * Fix normalization in GetSessions and ListSessions * Frontend renders using normalized identity info * Making Josh Happy :) * Showing previous use (not real data yet) * Run make format * RP and AP name fetching * Add test for GetAPDisplayName * Add staging/dev/prod credential for cross ecosystemw ork * Fix tests * Fix import issue from merge with main * Fix build errors * Apply formatting changes * Fix build errors * Add staging/prod idv templates * Use incredibly beautiful json encoding literalness * Refactor to external session functions * Apply formatting changes * Update packages/dotnet/Trinsic.Shared/Services/IDV/IDVSessionService.cs Co-authored-by: Joshua Coffey [email protected] * Correctly parse credential data * Apply formatting changes --------- Co-authored-by: Jan-Pieter George [email protected] Co-authored-by: Jan-Pieter George [email protected] Co-authored-by: Joshua Coffey [email protected] Co-authored-by: Mewmba [email protected]