Skip to content
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

Docs and typedoc integration #38

Merged
merged 3 commits into from
Nov 25, 2024
Merged

Docs and typedoc integration #38

merged 3 commits into from
Nov 25, 2024

Conversation

lovesh
Copy link
Member

@lovesh lovesh commented Nov 18, 2024

No description provided.

@lovesh lovesh mentioned this pull request Nov 18, 2024
README.md Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
useDefaults: boolean;
defaultMinimumInteger: number;
defaultMinimumDate: number;
defaultDecimalPlaces: number;
}

export const DefaultSchemaParsingOpts: ISchemaParsingOpts = {
useDefaults: false,
useDefaults: false,
Copy link
Contributor

Choose a reason for hiding this comment

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

Is the space here at the beginning of the line intended?

Copy link
Member Author

Choose a reason for hiding this comment

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

No.

@@ -20,10 +20,10 @@ export class Responses extends BytearrayWrapper {}

export class HashedKeys extends BytearrayWrapper {}

export class Participant {
// Id of this participant
export class BaseOTParticipant {
Copy link
Contributor

Choose a reason for hiding this comment

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

This is probably my lack of understanding here, but I don't see this class BaseOTParticipant used anywhere in the tests or in other places in the code.

Copy link
Member Author

Choose a reason for hiding this comment

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

In threshold sig test.

Copy link
Contributor

Choose a reason for hiding this comment

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

Now, I see. thanks! 👍

@lanterno
Copy link
Contributor

@lovesh Great PR! and special thanks for all the documentation on the anonymous credentials. I didn't manage to go through it all, but I'll take some time later to try the examples.

Signed-off-by: lovesh <[email protected]>
@lovesh lovesh merged commit 82d19b6 into master Nov 25, 2024
13 checks passed
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