Skip to content

[SG-800] Selecting notification for inactive account on iOS while app is open does not display the request#2166

Merged
andrebispo5 merged 2 commits intomasterfrom
SG-800-notification-tap-lock
Nov 4, 2022
Merged

[SG-800] Selecting notification for inactive account on iOS while app is open does not display the request#2166
andrebispo5 merged 2 commits intomasterfrom
SG-800-notification-tap-lock

Conversation

@andrebispo5
Copy link
Contributor

Type of change

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

Objective

Fix bug where If the user never answers the switch accounts pop up, the lock is never released. This was problematic because when tapping a notification while pop-up was showing, the user navigates to the other account automatically and the pop-up is never dismissed.

Code changes

Removed await to allow the execution flow and lock to be released.

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

If the user never answers the pop up, the lock is never released.
@andrebispo5 andrebispo5 requested a review from a team November 4, 2022 11:03
@andrebispo5 andrebispo5 requested a review from fedemkr November 4, 2022 16:22
@andrebispo5 andrebispo5 merged commit 69f02ee into master Nov 4, 2022
@andrebispo5 andrebispo5 deleted the SG-800-notification-tap-lock branch November 4, 2022 16:54
andrebispo5 added a commit that referenced this pull request Nov 4, 2022
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