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

fix: PIN validation passing with incorrect PIN #957

Merged
merged 3 commits into from
Sep 6, 2023

Conversation

jp0707
Copy link
Contributor

@jp0707 jp0707 commented Sep 5, 2023

Summary of Changes

Fix PIN validation passing with incorrect PIN.

Related Issues

#912

Pull Request Checklist

  • All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
  • Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
  • Updated documentation as needed for changed code and new or modified features;
  • Added sufficient tests so that overall code coverage is not reduced.

@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #957 (8e067c9) into main (01cb184) will increase coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main     #957      +/-   ##
==========================================
+ Coverage   59.26%   59.28%   +0.02%     
==========================================
  Files         176      176              
  Lines        4981     4981              
  Branches     1399     1399              
==========================================
+ Hits         2952     2953       +1     
+ Misses       2004     2003       -1     
  Partials       25       25              
Files Changed Coverage Δ
...ges/legacy/core/App/components/inputs/PINInput.tsx 80.00% <50.00%> (+4.00%) ⬆️

@bryce-mcmath bryce-mcmath changed the title Fix PIN validation passing with incorrect PIN. fix: PIN validation passing with incorrect PIN Sep 6, 2023
Copy link
Contributor

@bryce-mcmath bryce-mcmath left a comment

Choose a reason for hiding this comment

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

One small change, otherwise good catch!

packages/legacy/core/App/components/inputs/PINInput.tsx Outdated Show resolved Hide resolved
packages/legacy/core/App/components/inputs/PINInput.tsx Outdated Show resolved Hide resolved
Signed-off-by: Jyoti Patel <jyotipatel152@gmail.com>
Signed-off-by: Jyoti Patel <jyotipatel152@gmail.com>
Signed-off-by: Jyoti Patel <jyotipatel152@gmail.com>
@bryce-mcmath bryce-mcmath merged commit 5a4dd5e into openwallet-foundation:main Sep 6, 2023
4 of 5 checks passed
@jp0707 jp0707 deleted the fix/invalidPIN branch September 6, 2023 23:21
wadeking98 pushed a commit to wadeking98/aries-mobile-agent-react-native that referenced this pull request Sep 8, 2023
…#957)

Signed-off-by: Jyoti Patel <jyotipatel152@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.

None yet

2 participants