-
Notifications
You must be signed in to change notification settings - Fork 9
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
ABW-2077 Sign Proof biometric authentication requires 2x biometric scan #695
Conversation
Sources/Features/DappInteractionFeature/Coordinator/DappInteractionFlow.swift
Outdated
Show resolved
Hide resolved
Merge only after @umair-rdx confirms that it works fine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Free to merge after testing proves it works. But might be hard until RCnetv3 is merged in here, because Grape Dashboard is broken, cannot send proper Persona data requests, and @xstelea is working on a fix for RCNetv3 Dashboard, see https://rdxworks.slack.com/archives/C03JJFEBR0V/p1693416811136369?thread_ts=1693407582.814709&cid=C03JJFEBR0V
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore today, irrelevant, but change to clock
later on please :)
Sources/Features/DappInteractionFeature/Coordinator/DappInteractionFlow.swift
Outdated
Show resolved
Hide resolved
82238c8
to
49550bb
Compare
Sources/Features/DappInteractionFeature/Coordinator/DappInteractionCoordinator+View.swift
Outdated
Show resolved
Hide resolved
Sources/Features/DappInteractionFeature/Coordinator/DappInteractionCoordinator+View.swift
Outdated
Show resolved
Hide resolved
Sources/Features/DappInteractionFeature/Coordinator/DappInteractionFlow.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BEFORE MERGE: Test throughly :)
good that you fixed some other incorrect usages of Task. I think we wanna bump the 100 ms slightly though
Jira ticket: ABW-2077
Notes
Purely SwiftUI issue. 0.1s seems to be good enough. It would be good to see if iOS 17 fixes that issue.
Video recording
Screen.Recording.2023-09-05.at.11.25.22.mov
PR submission checklist