Skip to content

Remove references to forceRevocationCheck#370

Merged
cykoder merged 1 commit intomasterfrom
cleanup/force-revocation-check
Oct 12, 2023
Merged

Remove references to forceRevocationCheck#370
cykoder merged 1 commit intomasterfrom
cleanup/force-revocation-check

Conversation

@cykoder
Copy link
Copy Markdown
Member

@cykoder cykoder commented Oct 6, 2023

No description provided.

@cykoder cykoder requested review from lovesh and olegnn October 6, 2023 14:52
Signed-off-by: Sam Hellawell <sshellawell@gmail.com>
@cykoder cykoder force-pushed the cleanup/force-revocation-check branch from 5a635fc to 2784147 Compare October 6, 2023 14:56
* @property {DIDResolver} [resolver] - Resolver to resolve the issuer DID (optional)
* @property {Boolean} [unsignedPresentation] - Whether to verify the proof or not
* @property {Boolean} [compactProof] - Whether to compact the JSON-LD or not.
* @property {Boolean} [forceRevocationCheck] - Whether to force revocation check or not.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We removed this in favour of skipRevocationCheck. Another missing option is skipSchemaCheck. See here for details. Can you please add those docs here? Thanks

Copy link
Copy Markdown
Member Author

@cykoder cykoder Oct 12, 2023

Choose a reason for hiding this comment

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

will spin off an issue and make a separate PR when have a bit of spare time - #372

@cykoder cykoder merged commit c60add8 into master Oct 12, 2023
@cykoder cykoder deleted the cleanup/force-revocation-check branch October 12, 2023 19:15
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