-
Notifications
You must be signed in to change notification settings - Fork 55
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
Improve NFC reading UX on android #130
Comments
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedHello, my name is Bernal. I am a software developer from Costa Rica. Currently, I work as a back-end developer. In my previous job, I worked as a full-stack developer. Personally, I enjoy developing both the back end and the front end. I have 4 years of experience in the software development industry. Some technologies I have worked with include: Typescript, React, C#, and Node. How I plan on tackling this issueTo reach the solution, the first step is to understand how the project works. This involves familiarizing yourself with the project's documentation, structure, and codebase. Next, thoroughly review the issue description to grasp the problem fully. Based on this understanding, brainstorm and propose a viable solution. Once you have a clear plan, implement the solution in the code. After coding, conduct comprehensive tests to ensure the fix resolves the issue without introducing new problems. Finally, review your changes and prepare them for submission, ensuring all aspects of the issue have been addressed effectively. |
@BernalHQ can you give me a human written version of how you plan to tackle the issue? |
Hi @remicolin. yes a can. First, I took a time to get the enough knowledge of how the project works. Then, i going to identify where android vibration trigger and do some logic to implement a animition that tell the user that stop moving the passport. Probaby this animation goint to be inspired in the IOS animation. Here, in the animation, probable ask you, if the animation has to be the same as IOS system. Then i goint to do some test to proof the animation. and that´s it. I'd really love to work on this issue. |
@BernalHQ sweet, you can start working on it! |
Hi @remicolin! |
Hi @remicolin, thanks for the oportunity. I going to start work it on it. And yes, please provide me a example of the IOS version animation. |
improve NFC reading UX on android #130
Is this fixed? I can't scan either of my passports on Android. Using a Pixel 8. |
Issue Description:
when users scan their passport using NFC on Android, the following steps occur:
Problem:
the vibration alone is not sufficient to inform users that they need to stop moving the phone and that the phone is currently reading the chip.
Request:
we would like an implementation similar to the one on iOS, with an animation showing the progress of the reading and displaying which datagroup is currently being read.
The text was updated successfully, but these errors were encountered: