Skip to content
This repository has been archived by the owner on Dec 12, 2024. It is now read-only.

Added manual overrides for credential issuance upon application review. #250

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

andresuribe87
Copy link
Contributor

Overview

When a review happens, the issuer admin might want to override some values that will be put in the credentials that will be produced.

Description

This PR changes. Noteworthy: using a map instead of a list for the overrides. This is because the order of the overrides doesn't matter.

How Has This Been Tested?

Added overrides for a credential in existing tests. Additionally modified the e2e tests.

References

Still part of SIP5

@@ -98,6 +100,7 @@ func (s Service) buildCredentialResponse(
return nil, nil, err
}
}
s.applyRequestData(&credentialRequest, credentialOverrides, od)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

methods without return values make me nervous!

@andresuribe87 andresuribe87 merged commit 06aa661 into TBD54566975:main Jan 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants