-
Notifications
You must be signed in to change notification settings - Fork 9
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
ποΈ Remove v1 protocol version and default to v2 #902
Conversation
ff137
commented
Jul 24, 2024
•
edited
Loading
edited
- π₯ removes the v1 issuer and verifier services
- π¨ removes "protocol version" from request bodies, as we now only support v2
- β update tests to accommodate to these changes
|
398542b
to
505363b
Compare
6190a3e
to
023250e
Compare
K8s Regression Test Coverage
|
β¦m Verifier facade
bb7ed60
to
db08905
Compare
K8s Test Coverage
|
K8s Regression Test Coverage
|
K8s Test Coverage
|
K8s Regression Test Coverage
|
Compacted with `https://jsonformatter.org/json-pretty-print` and then formatted with `prettier`
|
K8s Test Coverage
|
K8s Regression Test Coverage
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - We should just consider how this would affect an existing wallet with a credential issued with the v1 protocol... π€