Skip to content

feat: add fetchEncryptedPrivKeys.ts - #2080

Merged
DariusParvin merged 1 commit into
masterfrom
BG-43925-fetch-keys-for-external-signer
Mar 8, 2022
Merged

feat: add fetchEncryptedPrivKeys.ts#2080
DariusParvin merged 1 commit into
masterfrom
BG-43925-fetch-keys-for-external-signer

Conversation

@DariusParvin

Copy link
Copy Markdown
Contributor

This file will help create a json with the users wallet ids and encrypted
private keys to be used by the external signer.

I created this file based off the files in modules/core/example.

Ticket: BG-43925

@DariusParvin
DariusParvin force-pushed the BG-43925-fetch-keys-for-external-signer branch from 168d1f2 to 60733aa Compare March 7, 2022 18:06
@codecov

codecov Bot commented Mar 7, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2080 (7924b51) into master (9a29d1f) will increase coverage by 87.30%.
The diff coverage is 100.00%.

❗ Current head 7924b51 differs from pull request most recent head 136fbab. Consider uploading reports for the commit 136fbab to get more accurate results

@@             Coverage Diff             @@
##           master    #2080       +/-   ##
===========================================
+ Coverage        0   87.30%   +87.30%     
===========================================
  Files           0      183      +183     
  Lines           0     9862     +9862     
  Branches        0     1419     +1419     
===========================================
+ Hits            0     8610     +8610     
- Misses          0      814      +814     
- Partials        0      438      +438     
Flag Coverage Δ
unit 87.30% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
modules/statics/src/networks.ts 100.00% <100.00%> (ø)
modules/account-lib/src/coin/xtz/transaction.ts 85.71% <0.00%> (ø)
...nt-lib/src/coin/sol/walletInitializationBuilder.ts 97.14% <0.00%> (ø)
modules/account-lib/src/coin/dot/transaction.ts 88.75% <0.00%> (ø)
...ules/account-lib/src/coin/sol/tokenEncodeDecode.ts 80.00% <0.00%> (ø)
...es/account-lib/src/coin/rbtc/transactionBuilder.ts 81.81% <0.00%> (ø)
...odules/account-lib/src/coin/celo/stakingBuilder.ts 93.33% <0.00%> (ø)
modules/account-lib/src/coin/etc/resources.ts 100.00% <0.00%> (ø)
modules/account-lib/src/coin/xtz/utils.ts 96.29% <0.00%> (ø)
modules/account-lib/src/coin/trx/address.ts 100.00% <0.00%> (ø)
... and 174 more

@DariusParvin
DariusParvin force-pushed the BG-43925-fetch-keys-for-external-signer branch 3 times, most recently from 20e83fe to aad3ea6 Compare March 7, 2022 20:14

@tylerlevine tylerlevine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

more good work here - just some suggestions for a little bit of a cleaner implementation and doc improvements.

Comment thread modules/express/src/fetchEncryptedPrivKeys.ts Outdated
Comment thread modules/express/src/fetchEncryptedPrivKeys.ts Outdated
Comment thread modules/express/src/fetchEncryptedPrivKeys.ts Outdated
This file will help create a json with the users wallet ids and encrypted
private keys to be used by the external signer.

Ticket: BG-43925
@DariusParvin
DariusParvin force-pushed the BG-43925-fetch-keys-for-external-signer branch from aad3ea6 to 136fbab Compare March 7, 2022 21:29
Comment thread modules/express/src/fetchEncryptedPrivKeys.ts
@DariusParvin

Copy link
Copy Markdown
Contributor Author

ready for re-review

@tylerlevine tylerlevine left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice job

@DariusParvin
DariusParvin merged commit 7f08e16 into master Mar 8, 2022
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