You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
After a burst of work in 3Q24 the rainbow_apk and rainbow_upgrade branches were left in a "demo" or "proof-of-concept" state with several new features.
This issue is an attempt to capture some of the know shortcomings of that sprint before memory fades
Send Expert Screen - account search
In contrast to the Send Basic screen, here the account name text is simply checked for existence on the blockchain. The multi-pronged lookup would be better. Q: is there enough screen layout space?
Account lookup should trim trailing space (or more generally, sanitize) before checking
Send Abroad screen
Available balance fiat equivalent does not work (dummy value is hard coded for demo); but fiat equiv does work in "proxy send". -- perhaps revise SwapEnterDataScreen to use Form widget containing both amount widgets, and cleaning up the GlobalKey-pushText action
Proxy Send confirmation screen
Should show both accounts (only shows recipient)
Send Expert Screen - signers list popup
Should show multisig weight for each signer
Should show threshold (and provide UI to change?)
Add New Authorization screen
Account name is not validated. Should use account search UI
Should be able to set multisig weight (use case: temporarily authorize specific single signer for a 2/3 msig)
Send Abroad fiat/token switch
does not update exchanged token balance
Fiat Exchange rate
Is it necessary to do all those ugly calls to rate routine?
Future: access valuation hook in Rainbow token contract
The text was updated successfully, but these errors were encountered:
After a burst of work in 3Q24 the
rainbow_apk
andrainbow_upgrade
branches were left in a "demo" or "proof-of-concept" state with several new features.This issue is an attempt to capture some of the know shortcomings of that sprint before memory fades
Send Expert Screen - account search
In contrast to the Send Basic screen, here the account name text is simply checked for existence on the blockchain. The multi-pronged lookup would be better. Q: is there enough screen layout space?
Account lookup should trim trailing space (or more generally, sanitize) before checking
Send Abroad screen
Available balance fiat equivalent does not work (dummy value is hard coded for demo); but fiat equiv does work in "proxy send". -- perhaps revise SwapEnterDataScreen to use Form widget containing both amount widgets, and cleaning up the GlobalKey-pushText action
Proxy Send confirmation screen
Should show both accounts (only shows recipient)
Send Expert Screen - signers list popup
Should show multisig weight for each signer
Should show threshold (and provide UI to change?)
Add New Authorization screen
Account name is not validated. Should use account search UI
Should be able to set multisig weight (use case: temporarily authorize specific single signer for a 2/3 msig)
Send Abroad fiat/token switch
does not update exchanged token balance
Fiat Exchange rate
Is it necessary to do all those ugly calls to rate routine?
Future: access valuation hook in Rainbow token contract
The text was updated successfully, but these errors were encountered: