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

feat(sdk-coin-coredao): add coredao sdk skeleton #5121

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

adarsh312
Copy link
Contributor

Ticket: WIN-3696

@adarsh312 adarsh312 force-pushed the WIN-3696_coredao_skeleton branch from a940798 to c92a1b7 Compare November 12, 2024 13:10
@adarsh312 adarsh312 marked this pull request as ready for review November 13, 2024 03:59
@adarsh312 adarsh312 requested review from a team as code owners November 13, 2024 03:59
@adarsh312 adarsh312 force-pushed the WIN-3696_coredao_skeleton branch from c92a1b7 to e8c0b55 Compare November 13, 2024 04:34
modules/sdk-coin-coredao/src/coredao.ts Outdated Show resolved Hide resolved
modules/sdk-coin-coredao/src/lib/index.ts Show resolved Hide resolved
modules/sdk-coin-coredao/src/lib/resources.ts Outdated Show resolved Hide resolved
modules/sdk-coin-coredao/src/lib/resources.ts Outdated Show resolved Hide resolved
modules/sdk-coin-coredao/src/lib/resources.ts Outdated Show resolved Hide resolved
modules/sdk-coin-coredao/test/resources/coredao.ts Outdated Show resolved Hide resolved
@adarsh312 adarsh312 force-pushed the WIN-3696_coredao_skeleton branch 5 times, most recently from 2e61f3e to 1b982fb Compare November 14, 2024 11:45
@adarsh312 adarsh312 requested a review from a team as a code owner November 14, 2024 11:45
@adarsh312 adarsh312 force-pushed the WIN-3696_coredao_skeleton branch 3 times, most recently from 890bddf to 9f3424d Compare November 15, 2024 06:37
gianchandania
gianchandania previously approved these changes Nov 15, 2024
@adarsh312 adarsh312 force-pushed the WIN-3696_coredao_skeleton branch from eaa1897 to 0e6ce18 Compare November 18, 2024 03:36

/** @inheritDoc */
getMPCAlgorithm(): MPCAlgorithm {
return 'ecdsa';
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: do we not have some type of const that can be used instead of the hardcoded string?

Copy link
Contributor

@sumanmukherjee03 sumanmukherjee03 left a comment

Choose a reason for hiding this comment

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

approving the codeowners change

@adarsh312 adarsh312 merged commit 0031451 into master Nov 18, 2024
9 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.

4 participants