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

NEVISACCESSAPP-5966: added password authenticator with custom policy #41

Merged

Conversation

daniel-toth-leeder
Copy link
Contributor

  • Changed inputs of PinScreen to be secure.
  • Fixed iOS only issue of keyboard hiding buttons.

- Changed inputs of PinScreen to be secure.
- Fixed iOS only issue of keyboard hiding buttons.
@daniel-toth-leeder daniel-toth-leeder force-pushed the feature/NEVISACCESSAPP-5966-add-password-authenticator branch from d25f666 to 7e13643 Compare July 17, 2024 12:44
@balazs-gerlei balazs-gerlei changed the title NEVISACCESSAPP-5966: updated sdk to 3.7.0 NEVISACCESSAPP-5966: added password authenticator with custom policy Jul 17, 2024
Copy link
Contributor

@balazs-gerlei balazs-gerlei left a comment

Choose a reason for hiding this comment

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

Let's discuss my one real comment (the other one is a compliment) and let's decide whether it warrants a change or not, I'm not choosing a review result until that!

Comment on lines -287 to +300
await startPinChange(eligibleAccounts);
await startPinChange(eligibleAccounts.at(0)!);
} else {
// in case that there are multiple eligible accounts then we have to show the account selection screen
return selectAccount(OperationType.pinChange);
}

async function startPinChange(accounts: Array<Account>) {
async function startPinChange(account: Account) {
const client = ClientProvider.getInstance().client;
client?.operations.pinChange
.username(accounts.at(0)!.username)
.username(account.username)
Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏻 Great refactor!

src/userInteraction/PasswordPolicyImpl.ts Outdated Show resolved Hide resolved
@daniel-toth-leeder daniel-toth-leeder force-pushed the feature/NEVISACCESSAPP-5966-add-password-authenticator branch from 7e13643 to 7791359 Compare July 18, 2024 12:30
@daniel-toth-leeder daniel-toth-leeder force-pushed the feature/NEVISACCESSAPP-5966-add-password-authenticator branch from 7791359 to cc4dcce Compare July 18, 2024 12:42
@daniel-toth-leeder daniel-toth-leeder merged commit 9f15efc into main Jul 18, 2024
6 checks passed
@daniel-toth-leeder daniel-toth-leeder deleted the feature/NEVISACCESSAPP-5966-add-password-authenticator branch July 18, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants