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

chore: export wallet-service api module #508

Merged
merged 1 commit into from
Apr 24, 2023

Conversation

luislhl
Copy link
Contributor

@luislhl luislhl commented Apr 18, 2023

Acceptance Criteria

  • Export the src/wallet/api/walletApi.ts module as walletServiceApi in the lib exports.

Security Checklist

  • Make sure you do not include new dependencies in the project unless strictly necessary and do not include dev-dependencies as production ones. More dependencies increase the possibility of one of them being hijacked and affecting us.

@luislhl luislhl requested a review from pedroferreira1 as a code owner April 18, 2023 18:43
@luislhl luislhl requested a review from andreabadesso April 18, 2023 18:44
@luislhl luislhl self-assigned this Apr 18, 2023
@codecov
Copy link

codecov bot commented Apr 18, 2023

Codecov Report

Merging #508 (293ad4c) into dev (7a7fcec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #508   +/-   ##
=======================================
  Coverage   55.88%   55.88%           
=======================================
  Files          66       66           
  Lines        5005     5005           
  Branches     1028     1028           
=======================================
  Hits         2797     2797           
  Misses       2113     2113           
  Partials       95       95           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@luislhl luislhl removed the request for review from pedroferreira1 April 20, 2023 15:10
@luislhl luislhl requested a review from pedroferreira1 April 20, 2023 15:31
@luislhl luislhl force-pushed the chore/export-wallet-service-api branch 2 times, most recently from 89bcbad to 293ad4c Compare April 24, 2023 15:12
@luislhl luislhl merged commit 11170b0 into dev Apr 24, 2023
@luislhl luislhl deleted the chore/export-wallet-service-api branch April 24, 2023 15:39
r4mmer added a commit that referenced this pull request May 18, 2023
* feat: validate user provided change address (#510)

* feat: validate user provided change address

* tests: getChangeAddress test

* Update src/storage/storage.ts

Co-authored-by: Tulio Miranda <tulio.mir@gmail.com>

---------

Co-authored-by: Tulio Miranda <tulio.mir@gmail.com>

* feat: start generating types when building (#482)

* chore: export wallet-service api module (#508)

* feat: new method to delete a tx-proposal in wallet-service api (#514)

* fix: allow providing pinCode when sending transaction with HathorWalletServiceWallet (#515)

* fix: allow providing pinCode when sending transaction with HathorWalletServiceWallet

* feat: allow a pinCode to be sent in the old facade sendTransaction method

* feat: new token utils (#516)

* feat: new token utils

* feat: generate access data with account path key (#512)

* feat: generate access data with account path key

* feat: leveldb dir name (#518)

* chore: let access data persist through stops

* feat: allow leveldb store on generic dir name

* check pin method (#513)

* feat: check pin and password methods

* feat: wallet id util (#523)

* fix created leveldb dir name (#522)

* fix: fix test directory name

* tests: use walletId util

* chore: unnecessary comment

* chore: bump version v1.0.0-rc3 (#528)

---------

Co-authored-by: Tulio Miranda <tulio.mir@gmail.com>
Co-authored-by: Luis Helder <luislhl@gmail.com>
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.

3 participants