Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion RELEASE-NOTES.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
2.38
-----

* Fixed back button navigation during manual login to return to the login screen instead of the home screen [#1751](https://github.com/Automattic/simplenote-android/issues/1751)

2.37
-----
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -93,8 +93,6 @@ open class NewCredentialsActivity : ThemedAppCompatActivity() {

override fun onBackPressed() {
super.onBackPressed()
this.startActivity(Intent(this, SimplenoteAuthenticationActivity::class.java))
finish()
}

@SuppressLint("RestrictedApi")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,12 @@ class SignInFragment: MagicLinkableFragment() {

private var authService: AuthorizationService? = null

private var manualLoginLauncher = registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
if (result.resultCode == Activity.RESULT_OK) {
activity?.finish()
}
}

private var resultLauncher = registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->

val data = result.data
Expand Down Expand Up @@ -118,7 +124,7 @@ class SignInFragment: MagicLinkableFragment() {
}
manualLoginTextView.setOnClickListener {
val email = getEmailEditText()
showLoginWithPassword(activity, email?.text?.toString())
launchManualLogin(email?.text?.toString())
}
return view
}
Expand All @@ -133,7 +139,7 @@ class SignInFragment: MagicLinkableFragment() {
hideDialogProgress()
if (state.code == 429) {
val email = getEmailEditText()
showLoginWithPassword(activity, email?.text?.toString())
launchManualLogin(email?.text?.toString())
}
Toast.makeText(context, getString(state.messageRes), Toast.LENGTH_LONG).show()
}
Expand Down Expand Up @@ -166,9 +172,20 @@ class SignInFragment: MagicLinkableFragment() {
}
}

private fun launchManualLogin(username: String?) {
val intent = Intent(requireActivity(), NewCredentialsActivity::class.java)
intent.putExtra("EXTRA_IS_LOGIN", true)
if (!username.isNullOrBlank()) {
intent.putExtra(Intent.EXTRA_EMAIL, username)
intent.putExtra(NewCredentialsActivity.PREF_HIDE_EMAIL_FIELD, true)
}
manualLoginLauncher.launch(intent)
}

override fun onDestroy() {
super.onDestroy()
resultLauncher.unregister()
manualLoginLauncher.unregister()
authService?.dispose()
}

Expand Down
Loading