Apply hooks suggestion
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Nov 29 2024
Add debug log to print error
Rebase before landing
Nov 28 2024
Nice, thanks!
Can we restyle the restore screen to look something like the registration navigator screens?
I'm not sure about this. A user enters the restore flow by going through the QR screen, which is a bit inconsistent with the registration flow (e.g. font size, alignment). I think that it would be better to follow the QR screen design so that the user doesn't experience inconsistency. We can also update the QR screen design to be consistent with the registration flow - I think this is the best solution.
- Can we avoid using the LoggedOutModal style for any of the screens in the navigator? It feels stylistically inconsistent to push a new screen that looks like what appears below the stack navigator.
Yeah, we can do that. In that case, I'd need to introduce new components instead of wrapping the old one.
Nov 27 2024
Great work on this! The conversion from Reanimated 1 to Reanimated 2/3 appears perfect, and the changes between them are pretty unavoidable without introducing a bunch of unnecessary complexity
In D14049#391602, @tomek wrote:In D14049#391563, @ashoat wrote:Do we really need the migration? Won't it get merged with defaultState anyways?
In the past, we had some bugs caused by relying on the default state instead of the migration. They can happen in the following scenario:
- A new store is added without a migration
- A new migration is added that uses this store
- A user updates the app from before the 1st step
Then migration 2 fails.
Code looks good, but passing back for a screen recording... I'm not sure about the copy, and a recording would will help me iterate on it
Thanks for the updated video!
Move check to deep link context. Use the usingRestoreFlow constant
In D14049#391563, @ashoat wrote:Do we really need the migration? Won't it get merged with defaultState anyways?
Nov 26 2024
Do we really need the migration? Won't it get merged with defaultState anyways?
Rebase
Revert setting the flag
Create a new screen with the restore flow
Rebase
Nit: Names are now a bit divergent: pickleAccountToString for account-only, and storeAsB64 for account+sessions but return types differ too. IMO storeAsB64 is misleading because Persist type isn't base64.
Thanks for explaining your process!! That sounds very thorough :)