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

SDK - Avoid resetting login state when calling AuthenticationService.getLoginFlow #6094

Merged
merged 3 commits into from
Jun 2, 2022

Conversation

ouchadam
Copy link
Contributor

@ouchadam ouchadam commented May 18, 2022

Type of change

  • Feature
  • Bugfix
  • Technical - SDK change
  • Other :

Content

Fixes #6093

Avoids resetting the internal authentication state when calling getLoginFlow until the flow is successfully fetched.

Motivation and context

To allow the UX flow of defaulting to a valid homeserver, editing with an invalid url and then returning to the previously selected homeserver.

Screenshots / GIFs

BEFORE AFTER
ezgif-4-26b6c4bc54 ezgif-4-d8f5749e9e

Tests

  • Start the sign in process
  • Edit the homeserver url to an invalid url and press next
  • After seeing the error quit the edit server screen
  • Continue to sign in with the previously selected homeserver
  • Notice a crash

Tested devices

  • Physical
  • Emulator
  • OS version(s): 31

@github-actions
Copy link

Unit Test Results

122 files  ±0  122 suites  ±0   2m 13s ⏱️ -14s
208 tests ±0  208 ✔️ ±0  0 💤 ±0  0 ±0 
702 runs  ±0  702 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 1898b94. ± Comparison against base commit 9a28949.

Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

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

Thanks!

@ouchadam ouchadam added the PR-Small PR with less than 20 updated lines label May 18, 2022
@ouchadam ouchadam force-pushed the feature/adm/ftue-sign-in branch 2 times, most recently from a0fcef6 to 8c44c98 Compare May 25, 2022 09:30
Base automatically changed from feature/adm/ftue-sign-in to develop May 25, 2022 16:30
@ouchadam ouchadam force-pushed the feature/adm/ftue-avoid-resetting-login-state branch from 1898b94 to 8e8d38c Compare June 1, 2022 15:48
@ouchadam ouchadam marked this pull request as ready for review June 1, 2022 15:48
@ouchadam ouchadam requested review from a team, onurays and mnaturel and removed request for a team and mnaturel June 1, 2022 15:49
Copy link
Contributor

@onurays onurays left a comment

Choose a reason for hiding this comment

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

Checked the commit history to see if we have an edge case here, nothing suspicious. LGTM!

@ouchadam ouchadam enabled auto-merge June 1, 2022 16:20
@bmarty
Copy link
Member

bmarty commented Jun 2, 2022

(auto-merge would be nice if the CI was not stuck so often...)

FTR:
image

@bmarty bmarty disabled auto-merge June 2, 2022 21:11
@bmarty bmarty merged commit b6b487d into develop Jun 2, 2022
@bmarty bmarty deleted the feature/adm/ftue-avoid-resetting-login-state branch June 2, 2022 21:11
@github-actions
Copy link

github-actions bot commented Jun 2, 2022

Matrix SDK

Integration Tests Results:

  • [org.matrix.android.sdk.session]
    = passed=20 failures=0 errors=0 skipped=3
  • [org.matrix.android.sdk.account]
    = passed=3 failures=0 errors=0 skipped=2
  • [org.matrix.android.sdk.internal]
    = passed=23 failures=1 errors=0 skipped=1
  • [org.matrix.android.sdk.ordering]
    = passed=16 failures=0 errors=0 skipped=0
  • [org.matrix.android.sdk.PermalinkParserTest]
    = passed=2 failures=0 errors=0 skipped=0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
matrix-sdk PR-Small PR with less than 20 updated lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SDK] Avoid resetting internal state when calling getLoginFlow
3 participants