Skip to content

Conversation

@joragua
Copy link
Collaborator

@joragua joragua commented Mar 31, 2025

Related Issues

App: https://github.com/owncloud/enterprise/issues/7134

  • Add changelog files for the fixed issues in folder changelog/unreleased. More info here
  • Add feature to Release Notes in ReleaseNotesViewModel.kt creating a new ReleaseNote() with String resources (if required)

QA

@joragua joragua self-assigned this Mar 31, 2025
@joragua joragua changed the title [FIX] Biometrics crash at start [FIX] App crashes at start when biometric fail Mar 31, 2025
@joragua joragua changed the title [FIX] App crashes at start when biometric fail [FIX] App crashes at start when biometrics fail Mar 31, 2025
@joragua joragua force-pushed the fix/biometrics_crash_at_start branch from a78807b to 4bce200 Compare March 31, 2025 07:31
@joragua joragua marked this pull request as ready for review March 31, 2025 07:39
@joragua joragua requested a review from JuancaG05 March 31, 2025 07:39
Copy link
Contributor

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

Some comments here @joragua!

@joragua joragua force-pushed the fix/biometrics_crash_at_start branch from 5e09f8a to 6199d53 Compare April 1, 2025 09:15
@joragua joragua requested a review from JuancaG05 April 1, 2025 09:18
Copy link
Contributor

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

Some more comments here @joragua!

@joragua joragua force-pushed the fix/biometrics_crash_at_start branch from c53dd50 to 934a9a0 Compare April 1, 2025 10:28
Copy link
Contributor

@JuancaG05 JuancaG05 left a comment

Choose a reason for hiding this comment

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

LGTM now!! 👏

@joragua
Copy link
Collaborator Author

joragua commented Apr 1, 2025

QA Tests

Test 1: ✅

Initially, no faceid/fingerprint enrolled in device

  1. Enroll with fingerprint && enable biometric -> fingerprint asked in the app
  2. Go to Settings and enroll faceid -> fingerprint asked (prior to Face ID)
  3. Go to Settings and disable fingerprint -> no biometric asked, pattern/passcode instead, and snackbar error
  4. Go to Settings and disable Face ID -> Biometric not available, just pattern and passcode in the app

Test 2: ✅

  1. In device, enroll with only Face ID
  2. Install master version, and in app settings enable pattern/passcode with biometric -> app will crash
  3. Update to the current version -> no crash, error message and pattern/passcode instead

@jesmrec
Copy link
Collaborator

jesmrec commented Apr 1, 2025

Thanks for testing @joragua!!

Let's move this forward

@joragua joragua merged commit 0065268 into master Apr 1, 2025
7 checks passed
@joragua joragua deleted the fix/biometrics_crash_at_start branch April 1, 2025 11:55
@jesmrec jesmrec mentioned this pull request Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants