Conversation
Signed-off-by: joydeep049 <bntripathy123@gmail.com>
|
Thank you @joydeep049 For awesome work. I will review it. In the mean time, please check: and consider re-using the type defined here, to make it more clear and concise! |
|
I have already made changes throughout the codebase, reusing the cryptokeys array wherever it was used. Am I missing something? |
@joydeep049 : Please see the Merged PR: #219 This is how the type |
I think, there is some more remaining changes in the base CoMID library, you can address the usage their, I suppose using this PR.. |
|
Looks like this issue was already solved by PR 219. |
The
AuthorizedByfield inCoSWIDEvidenceMapandMeasurementwas initially only one cryptokey, but it should be an array of cryptokeys.This PR makes the above change and makes all subsequent required changes throughout the codebase.
Local Tests Passed ✔️
cc @thomas-fossati @yogeshbdeshpande @jraman567