balance-io / balance Goto Github PK

I think it would be worth understanding how we are using data from Infura and calling information. I just want to check it is as efficient as possible:

2022-02-28 09:09:23.022445-0800 Balance iOS[18987:6629295] [connection] nw_resolver_start_query_timer_block_invoke [C5] Query fired: did not receive all answers in time for mainnet.infura.io:443
2022-02-28 09:09:33.616096-0800 Balance iOS[18987:6629297] [connection] nw_resolver_start_query_timer_block_invoke [C12] Query fired: did not receive all answers in time for ropsten.infura.io:443
2022-02-28 09:10:00.105046-0800 Balance iOS[18987:6629999] [connection] nw_resolver_start_query_timer_block_invoke [C13] Query fired: did not receive all answers in time for kovan.infura.io:443
2022-02-28 09:10:04.438838-0800 Balance iOS[18987:6629835] [connection] nw_resolver_start_query_timer_block_invoke [C14] Query fired: did not receive all answers in time for rinkeby.infura.io:443
2022-02-28 09:10:04.751030-0800 Balance iOS[18987:6629835] [connection] nw_resolver_start_query_timer_block_invoke [C15] Query fired: did not receive all answers in time for goerli.infura.io:443

Current behaviour:

Go to NFT tab. Tap. Wait. Load.

Ideal behaviour:

The wallet is pre-loading the NFT tab in the background to make sure they are just there the second you go to the tab.

Today, I manually set my wallet’s ENS. Would be great if it auto-detected it and just set the name to that. People can change it, of course.

I attempted to bride .05 Eth L1 to L2 on arbitrum bridge and received the above error message when signing the transaction in my Balance wallet.

I attempted to bridge .05 Eth L1 to L2 on optimism bridge and received the above error message when signing the transaction in the wallet.

I attempted to swap .01 Eth for USDC on uniswap and received the above error message when signing the transaction in the wallet.

I attempted to swap .01 Eth for Sushi on sushi swap and received the above error message when signing the transaction in the wallet.

I pressed the "Reveal Secret Recovery Phrase" button and the wallet crashed as shown above.

We should try to take into account assets’ prices and amounts when presenting the numbers that comprise balances.

$3k ETH needs more decimal places for the amounts of the token than $0.48 GRT.

This feels like a good framework for when to truncate and when to expand on numbers.

The quick fix is to apply this to the Balances section inside the wallet screen.

The longer fix is a consistent approach to number formatting, fiat currency values, and token prices.

I like to use SF Mono Light for prepended currency symbols and everything after the decimal place.

I use SF Mono Medium for the meat of a number.

I just wanted to check what is required for adding Alchemy as a fallback resource for NFT data and Transaction broadcasting.

How can we do that elegantly and monitor if our endpoints are getting spammed?

Personally I hate them. They are bad for flow.

No one reads them.

Please can we remove it!

I need to research how this is possible.

This is a quick first pass at a QR code viewer in the prototype:

We need to add a button on the wallet view:

Note the new address format where the first and last 4 important address characters are BOLD and 100% opaque and the less important characters—including 0x—are at 80% opacity:

The QR code popover square should blur the background of the interface:

The floating X should make the QR code slide up and away. The circle should slide down.

I used this QR Code generator: https://www.qrcodechimp.com

It looks like I can't share the actual QR Code from my account but here are its settings if you want to explore:

Need to establish Swift Colors. Here are the hexes.
Eye Colour: #192A36
Stroke Colour: #03C78D

There are lots of Swift QR Code libraries. Finding a beautiful one is important to me. Happy to help explore.

##Research
https://medium.com/@dominicfholmes/generating-qr-codes-in-swift-4-b5dacc75727c

https://www.hackingwithswift.com/books/ios-swiftui/generating-and-scaling-up-a-qr-code

it would be nice to show the icons for each network

This feels unneeded:

Right now if someone has set Chrome to be their default iOS browser the app bounces them out to Chrome instead of Safari.

We need to look into sending users straight into Safari even if Chrome is their default.

Once in the wallet view it would be great if you could derive the other accounts.

This way we can have access to other accounts that may have been created in metamask or other wallets