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

Vc fields update #8

Merged
merged 3 commits into from
Nov 1, 2024
Merged

Vc fields update #8

merged 3 commits into from
Nov 1, 2024

Conversation

rolandgroen
Copy link
Collaborator

Fill the VC fields with the right properties for further processing in the NUTS trust infrastructure.

Updated the uraCredential function to use 'issuer' instead of 'did' for the issuer parameter and simplified the mapping of credential subjects. Also, standardized the constant naming for SAN_TYPE_OTHER_NAME to SanTypeOtherName across the codebase to ensure consistency.
* main:
  Refactor constant name SAN_TYPE_OTHER_NAME to SanTypeOtherName
@rolandgroen rolandgroen marked this pull request as ready for review November 1, 2024 12:19
@rolandgroen rolandgroen requested a review from reinkrul November 1, 2024 12:19
Moved UZI number validation to check before generating uraCredential, ensuring serialNumber matches UZI number early in the process. Also removed redundant serialNumber parameter from uraCredential function.
@rolandgroen rolandgroen merged commit 8366b22 into main Nov 1, 2024
3 checks passed
@rolandgroen rolandgroen deleted the vc_fields branch November 1, 2024 13:07
rolandgroen added a commit that referenced this pull request Nov 1, 2024
* main:
  Vc fields update (#8)

# Conflicts:
#	uzi_vc_issuer/ura_issuer.go
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