Skip to content

[SG-831] Pull Down Sync does not retrieve pending AuthRequests#2196

Merged
andrebispo5 merged 3 commits intomasterfrom
bugfix/SG-831-pull-to-refresh-requests
Nov 17, 2022
Merged

[SG-831] Pull Down Sync does not retrieve pending AuthRequests#2196
andrebispo5 merged 3 commits intomasterfrom
bugfix/SG-831-pull-to-refresh-requests

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

Adding the possibility to manually get pending login requests. This will be always fetched even though the sync is not forced.

Code changes

Exposed the Sync Login Requests method to be called separately from the FullSync. This is a temporary solution. A tech dept ticket was created to move all Passwordless code to an independent service.

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

@andrebispo5 andrebispo5 requested review from a team November 17, 2022 14:53
@andrebispo5 andrebispo5 merged commit 34fd30e into master Nov 17, 2022
@andrebispo5 andrebispo5 deleted the bugfix/SG-831-pull-to-refresh-requests branch November 17, 2022 16:02
trmartin4 pushed a commit that referenced this pull request Nov 17, 2022
* [SG-831] Pull to refresh forces refresh.

* [SG-831] Expose sync login request method to be used independently

* [SG-831] Change sync order

(cherry picked from commit 34fd30e)
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