search

animo / paradym-wallet

With Paradym Wallet, you can seamlessly manage and present your digital credentials
https://paradym.id
Apache License 2.0
32 stars 12 forks source link

feat: typography and optional pid #175

Closed janrtvld closed 1 month ago

janrtvld commented 1 month ago

I've restructured the typography so that it's more consistent. I've also made the PID optional during the onboarding. For now, it's disabled on the main screen and if you want to set it up you'll need to reset. I'm working on extracting the scanning logic out of the onboarding but that was a bit more work than i thought 🥲.

For now this removes the PID req so we can test easier with just regular EAA's.

https://github.com/user-attachments/assets/8eb46b4b-c7fc-4825-91f0-4037c75cd35b

TimoGlastra commented 1 month ago

Yeah the onboarding became quite complex sadly. How do you want to show it within the app on main screen, as a modal? Or a push screen?

Would be nice if we can reuse the complete flow, as otherwise we'll have consistencies very quickly 😬

janrtvld commented 1 month ago

It's quite some steps so I was thinking just a regular screen flow. I want to re-use most of it but there might be some differences. E.g. if you do it after the onboarding there is an extra screen needed for your wallet pin, as you can not re-use the value that the user has entered during the onboarding.

But will do my best 👍