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

check pin method #513

Merged
merged 16 commits into from
May 15, 2023
Merged

check pin method #513

merged 16 commits into from
May 15, 2023

Conversation

r4mmer
Copy link
Member

@r4mmer r4mmer commented Apr 20, 2023

Acceptance Criteria

  • Create a method for wallets to check the pin and password
  • Create a method for wallets to change the pin and password

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.

@r4mmer r4mmer requested a review from andreabadesso April 20, 2023 14:33
@r4mmer r4mmer self-assigned this Apr 20, 2023
@r4mmer r4mmer requested a review from pedroferreira1 as a code owner April 20, 2023 14:33
@codecov
Copy link

codecov bot commented Apr 20, 2023

Codecov Report

Merging #513 (a7c559b) into dev (b132413) will increase coverage by 0.12%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##              dev     #513      +/-   ##
==========================================
+ Coverage   58.14%   58.26%   +0.12%     
==========================================
  Files          66       66              
  Lines        5084     5106      +22     
  Branches     1049     1053       +4     
==========================================
+ Hits         2956     2975      +19     
- Misses       2040     2043       +3     
  Partials       88       88              
Impacted Files Coverage Δ
src/types.ts 33.33% <ø> (ø)
src/wallet/wallet.ts 30.15% <0.00%> (-0.17%) ⬇️
src/utils/wallet.ts 93.37% <91.30%> (-0.31%) ⬇️
src/new/wallet.js 34.60% <100.00%> (+0.34%) ⬆️
src/storage/storage.ts 59.78% <100.00%> (-0.82%) ⬇️
src/utils/crypto.ts 93.75% <100.00%> (+0.41%) ⬆️

@r4mmer r4mmer force-pushed the feat/check-pin-method branch from 5eeccaa to 2593d10 Compare April 20, 2023 15:52
@r4mmer r4mmer requested review from alexruzenhack and removed request for andreabadesso April 24, 2023 13:48
src/storage/storage.ts Outdated Show resolved Hide resolved
src/storage/storage.ts Show resolved Hide resolved
src/storage/storage.ts Show resolved Hide resolved
__tests__/utils/wallet.test.js Outdated Show resolved Hide resolved
__tests__/utils/wallet.test.js Show resolved Hide resolved
__tests__/utils/wallet.test.js Show resolved Hide resolved
__tests__/utils/wallet.test.js Show resolved Hide resolved
__tests__/utils/wallet.test.js Show resolved Hide resolved
__tests__/utils/wallet.test.js Show resolved Hide resolved
src/utils/wallet.ts Show resolved Hide resolved
src/utils/wallet.ts Outdated Show resolved Hide resolved
src/utils/wallet.ts Outdated Show resolved Hide resolved
src/new/wallet.js Show resolved Hide resolved
@r4mmer r4mmer force-pushed the feat/check-pin-method branch from b795e8e to 8629226 Compare April 28, 2023 18:57
@r4mmer r4mmer force-pushed the feat/check-pin-method branch 2 times, most recently from aaf5fe3 to a4e1bc0 Compare May 15, 2023 13:33
@r4mmer r4mmer force-pushed the feat/check-pin-method branch from a4e1bc0 to 4ae2d0a Compare May 15, 2023 13:45
@pedroferreira1 pedroferreira1 self-requested a review May 15, 2023 15:19
@r4mmer r4mmer merged commit 83e64f0 into dev May 15, 2023
@r4mmer r4mmer deleted the feat/check-pin-method branch May 15, 2023 15:38
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