Two concerns:
- Feed Queries
- All Stories
- Search
- Feed Search
- Transactions
- Transaction Logs
Jun 1 2023
JS looks good. Rust changes look simple but I'm not clear on how all of them connect to the JS. Might be good to add a reviewer who has context on that part of the codebase, but defer to @jon
May 31 2023
May 30 2023
displayFailureAlert
displayFailureAlert
Make setThreadAvatar return Promise<void> for consistency with D8017, since nothing needs its return anyways
Make setUserAvatar return Promise<void> so that it can be used in the registration flow with the expectation that it calls the keyserver
May 29 2023
Thanks for annotating everything. I tried it locally and couldn't figure out anything better, unfortunately
May 26 2023
This permission should not have been introduced without corresponding changes on the native side to prevent massive inconsistency reports from being generated.
Comment
What happens on a refresh? Wondering if the results get "replaced" after the load is complete, or if they get "merged"
Lack of syntax highlighting for Kotlin was annoying... solved it with this. Let me know if any other languages have similar problems