-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: Implimentation for revoking only when data in wallet changes #184
feat: Implimentation for revoking only when data in wallet changes #184
Conversation
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.
You shouldn't update the copyright year for existing files.
You'll need to also store the content hash after issuance, otherwise there will be no hash to compare when being revoked.
src/main/java/uk/nhs/hee/tis/trainee/credentials/dto/PlacementCredentialDto.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/dto/ProgrammeMembershipCredentialDto.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/event/PlacementEventListener.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/event/PlacementEventListener.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/event/ProgrammeMembershipEventListener.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/event/ProgrammeMembershipEventListener.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/service/RevocationService.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/service/RevocationService.java
Outdated
Show resolved
Hide resolved
src/main/java/uk/nhs/hee/tis/trainee/credentials/dto/PlacementCredentialDto.java
Outdated
Show resolved
Hide resolved
refactor: sonarcloud issues refactor: sonarcloud issues refactor: remove old hash method test: test for md5Hash refactor: seperate MD5 hash method into seperate utill file to decrease code duplication github comments and tests test: update tests feat: Github Comments
145af32
to
29752be
Compare
|
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.
🤞 😁
@EdwardBarclay are you set up for testing on stage? |
Nope I was just thinking about that before merging |
Rats. My staging access is also a bit broken at present (I think my identity credential is for the old server). Let's chat tomorrow and see if we can get something working... |
TICKET TIS21-6067