Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Desktop: Add two-factor authorisation to Password modal #764

Merged
merged 2 commits into from Dec 11, 2018

Conversation

rihardsgravis
Copy link
Contributor

Description

Add two-factor authorisation to Password modal, enabling 2fa authorisation at View Seed and Reset wallet views.

Fixes #642

Type of change

  • Enhancement (a non-breaking change which adds functionality)

How Has This Been Tested?

Tested manually on macOS

Checklist:

  • My code follows the style guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@rihardsgravis rihardsgravis added C - Desktop E - Security Epic - Security related labels Dec 10, 2018
@rihardsgravis rihardsgravis self-assigned this Dec 10, 2018
@rihardsgravis rihardsgravis added the L - Need reviews Lifecycle - Each PR must have at least two reviewers label Dec 10, 2018
@rihardsgravis rihardsgravis removed the L - Need reviews Lifecycle - Each PR must have at least two reviewers label Dec 11, 2018
@rihardsgravis rihardsgravis merged commit d3f3fe9 into develop Dec 11, 2018
@rihardsgravis rihardsgravis deleted the feat/view-seed-2fa branch December 11, 2018 13:29
rihardsgravis pushed a commit that referenced this pull request Dec 12, 2018
* develop: (141 commits)
  New Crowdin translations [ci skip] (#769)
  Mobile: Fix minor bugs (#771)
  Assign account index to accounts in state (#715)
  Mobile: Fix failing tests (#770)
  Desktop: Add two-factor authorisation to Password modal (#764)
  Localization improvements (#718)
  Mobile: Fix keyboard avoidance globally (#767)
  Mobile: Handle error if permissions are denied on SeedVault import (#719)
  New Crowdin translations [ci skip] (#667)
  Desktop: Delete yarn.lock (#768)
  Update Seed write down view `Done` locale to `Back` (#765)
  Mobile: Minor refactor
  Update src/mobile/android/gradlew.bat
  Mobile: Minor code cleanup
  Fix unresolved conflict
  Localise string
  Update src/mobile/src/ui/views/wallet/DeleteAccount.js
  Update src/mobile/src/ui/components/TransferConfirmationModal.js
  Preserve order of themes via indexes in ThemeCustomisation component (#763)
  [Security] Bump lodash from 4.17.4 to 4.17.10 (#723)
  ...

# Conflicts:
#	src/desktop/src/ui/views/account/Addresses.js
#	src/desktop/src/ui/views/settings/Advanced.js
#	src/desktop/src/ui/views/settings/account/Seed.js
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C - Desktop E - Security Epic - Security related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants