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

BankID NO: add a note about synthetic SSNs #115

Merged
merged 1 commit into from
May 15, 2024

Conversation

nmoskaleva
Copy link
Collaborator

The Biometrics app doesn't have support for synthetic SSNs.
So when generating a test user, the "Synthetic" box should be unchecked.

@nmoskaleva nmoskaleva requested a review from mickhansen May 15, 2024 07:53
Copy link

netlify bot commented May 15, 2024

Deploy Preview for criipto-docs ready!

Name Link
🔨 Latest commit b1dce9b
🔍 Latest deploy log https://app.netlify.com/sites/criipto-docs/deploys/66447bc4a0dd0b000853d074
😎 Deploy Preview https://deploy-preview-115--criipto-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

4. Click "Order" to initiate the process
5. Click the pencil icon and add a phone number and an email that you want to associate with the test user
1. Go to the ["TEST NUMBER GENERATOR"](https://ra-preprod.bankidnorge.no/#/generate) to generate a random, valid SSN.
_Please ensure that the "Synthetic" checkbox is unchecked before generating a new number. [BankID Biometric](/verify/e-ids/norwegian-bankid/#testing-bankid-biometric) app does not currently support synthetic SSN numbers, so you won't be able to use them with the app._
Copy link
Collaborator

Choose a reason for hiding this comment

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

synthethic works for non-biometrics login right? So perhaps it should be a conditional statement, "If you want to test biometrics, please make sure to"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mickhansen updated

@nmoskaleva nmoskaleva force-pushed the update-bankid-no-guide branch from 8f2847e to b1dce9b Compare May 15, 2024 09:09
@mickhansen mickhansen merged commit 255de94 into master May 15, 2024
5 checks passed
@mickhansen mickhansen deleted the update-bankid-no-guide branch May 15, 2024 09:21
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