Skip to content

[PM-1129] iOS 16 Third-Party 2FA OTP handling#2409

Merged
fedemkr merged 8 commits intomasterfrom
feature/EC-979-iOS-third-party-2fa
Mar 9, 2023
Merged

[PM-1129] iOS 16 Third-Party 2FA OTP handling#2409
fedemkr merged 8 commits intomasterfrom
feature/EC-979-iOS-third-party-2fa

Conversation

@fedemkr
Copy link
Member

@fedemkr fedemkr commented Mar 9, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

This adds OTP handling feature to iOS and adds Bitwarden app as a 2FA option for the iOS platform.

This PR is the combination of several other ones that have already been reviewed and approved:
#2370
#2404
#2407
#2408

Code changes

Check the other PRs aforementioned

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

* EC-980 added Bitwarden as otpauth scheme handler

* EC-980 Fix format
* EC-981 Started adding OTP to existing cipher. Reused AutofillCiphersPage for the cipher selection and refactored it so that we have more code reuse

* EC-981 Fix navigation on otp handling

* EC-981 Fix formatting

* EC-981 Added otp cipher selection callout and add close toolbar item when needed
@fedemkr fedemkr added the ios label Mar 9, 2023
@fedemkr fedemkr requested a review from a team March 9, 2023 00:31
@fedemkr fedemkr merged commit a18f74a into master Mar 9, 2023
@fedemkr fedemkr deleted the feature/EC-979-iOS-third-party-2fa branch March 9, 2023 14:16
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