This is almost the last update of Fiji. The one remaining is a redesign of ERC20, ERC721 tokens.

Zeplin:

1. Blank https://app.zeplin.io/project/5d088205bff2d15de6a4397b/styleguide/components?cseid=5e218a805b5b617d8eec7aef

2. Entering https://app.zeplin.io/project/5d088205bff2d15de6a4397b/styleguide/components?cseid=5e218c6773e393a79c640116

3. Entered https://app.zeplin.io/project/5d088205bff2d15de6a4397b/styleguide/components?cseid=5e218cbacf6a85ab00ee4d3b

4. Error https://app.zeplin.io/project/5d088205bff2d15de6a4397b/styleguide/components?cseid=5e218d1b0f5ab5ab7cdca668

Problem: users have to enter RPC networks manually, where in most cases it can be automated.

Changes:

• We add a second line in the list where we mention ChainID
• We add tabs in "Add Custom RPC Network". One is to Enter it manually, second to browse among all existing networks (we can use chainlist)
• Users should be able to hide any network they want except Ethereum Mainnet, but by default, we keep it as it is

Prototype: https://invis.io/MW1220T36ET3

https://user-images.githubusercontent.com/51817359/143016538-ab62a697-ff92-4d51-8896-46ea2c665f85.mov

Very hard to reproduce, since I am not doing anything special. Usually, the app crashes during my inactivity.

1. So I open the AlphaWallet app
2. I just play around
3. Then I leave it, coming back working on desktop
4. Then all of a sudden I see the info about the AlphaWallet crash
5. No idea what's going on

Version: 3.37 (421)

Observe this file https://github.com/AlphaWallet/TokenScript-Examples/blob/train-leaves/examples/action/ETH/ETH.xml go to line 71 and see the hardcoded name is depositeAAVE

-> only PR to train-leaves

Sketch Cloud: https://sketch.cloud/s/e4OJW/v/DP2LVa/a/oE3M0x/ https://sketch.cloud/s/e4OJW/v/DP2LVa/a/YP3Zwm

Zeplin: https://zpl.io/bWn60RP https://zpl.io/aX16JzE

• Display ENS reverse lookup if they correspond directly to ETH address
• when you click "Add" display native iOS menu
• Display Blockies
• Display balance
• Display ENS reverse lookup | address in a format 0x, four characters, ellipsis, four last characters

There are pending tasks for ERC1155 support after #3221:

• [ ] Classify token IDs in an ERC1155 contract into collections. i.e. 1 contract, N collections
  • James seems to have a solution
• [ ] Can we take decimals > 0 as meaning the token ID is fungible? (But can it be a fractional NFT?) Support batch transfers (and Select at top right hand corner of Assets sub-tab needs to be enabled)
• [x] Looks like a bug: let view = SingleTokenCardAmountSelectionToolbarView(viewModel: .init(availableAmount: 20, selectedAmount: 1))
  • Affects ERC721 ? Beause it's not used for ERC1155
• [ ] Missing fields in token collection. Not sure we can get all of them (refer to UI design)
  • Issuer
  • Assets
  • Melts
  • Hodlers/Holders
  • Transfers
• [x] Missing fields in token ID screen (refer to UI design)
  • Issuer
  • Circulating Supply
  • Total Supply
  • Reserve Value
  • Type
  • Transferable
  • Melt Fee
  • Transfer Fee
  • Created (i.e. date)
• [ ] Some token URI JSONs have more metadata (e.g. properties?) eg. https://lostrelics.io/item-metadata/18000000000005e1000000000000000000000000000000000000000000000000
• [ ] Check add custom token from UI handles ERC1155 correctly. Especially balance. Also name/symbol is not part of ERC1155 (e.g. Enjin), and symbol is not applicable
• [ ] Check if relevant, about how tokenURI is interpreted: https://github.com/AlphaWallet/AlchemyNFT/issues/513
• [x] Tap token row in send 1155 should expand to show each tokenId + value
• [x] The big image after tapping on a token (tokenId) should almost fill the screen horizontally
• [x] Show default/fallback token ID icon when they are not available #3223

Fixes #3205

I have a possible fix. Can @oa-s and @hboon test to see if this works?

make clean && make install_bundle && make bootstrap && make release

Note (1-old):

I reduced the Fastfile to the following lines below and I was able to proceed until the upload to App Store stage (which I cut short by hitting control-c). I then retrofitted the line back into the original Fastfile.

default_platform(:ios)

platform :ios do
  desc "Push a new release build to the App Store"
  lane :release do
    increment_build_number(xcodeproj: "AlphaWallet.xcodeproj")
    build_app(workspace: "AlphaWallet.xcworkspace", scheme: "AlphaWallet", xcargs: "-allowProvisioningUpdates")
    upload_to_app_store(skip_metadata: true, skip_screenshots: true, skip_binary_upload: true)
  end
end

Note (2-version numbers):

Somehow the fastlane script was expecting strings in "x.y.z" in the Info.plist files but the Info.plist files had expressions ($CURRENT_PROJECT) etc which was confusing it. The version number has to be in the "x.y.z" form or the fastlane script just barfs.

We can get this additional data for NFTs available on OpenSea:

{
  "stats": {
    "count": 1700,
    "num_owners": 596,
    "floor_price": 4.9
    "total_volume": 9397.148721515618,
    "average_price": 9.159014348455768,

    "total_sales": 1026,
    "total_supply": 1700,
    //…
}

Here's the corresponding OpenSea page showing some of these data:

Maybe we can display them in the Info tab:

But perhaps we can highlight the floor price?

I was thinking if we should use the floor price:

• display in the Wallet tab
• include in fiat value of the wallet

What do you think?

Notes for implementation:

Doc: https://docs.opensea.io/reference/retrieving-collection-stats

//Get tokens owned by user:
curl -H 'X-API-KEY: <key>' 'https://api.opensea.io/api/v1/assets/?owner=0xbbce83173d5c1D122AE64856b4Af0D5AE07Fa362&order_by=pk&order_direction=asc&limit=50&offset=0'

//Read slug
"slug" : "blitmap"

//Use slug
curl -H 'X-API-KEY: <key>' 'https://api.opensea.io/api/v1/collection/blitmap/stats'

Like in https://zpl.io/bJmwEox. Can you see first if it can be done in a way that is not too fragile or hard to maintain?

Note that users can set a "local wallet name" which overrides the ENS name. in Settings > Change Wallet > Tap and hold a wallet.

When trying to connect to the Polygon version of Uniswap or Anyswap via WC on iOS I get this.

Next, I enable Ethereum just to allow WC to connect, but the app is then locked on Ethereum mainnet. Clicking the 'switch to polygon' in the dapp doesn't do anything (press, no change in the network on Uniswap, Prompt - 'switch Metamask to Polygon' on Anyswap.

This has started after the latest update.

On Android, everything works fine as it does allow you to pick up which chainID you are connecting using WalletConnect, before confirming.

Part of #3546 Depends on: #3580

A token matches if the contract and chain ID matches an entry in each array.

This file: https://github.com/AlphaWallet/tokens/blob/main/tokens.json (check update: https://github.com/AlphaWallet/alpha-wallet-ios/issues/3581#issuecomment-1019835025)

"All" should contain every token that is not hidden (like it is before the PR).

If a token is not categorized, display it under "Assets".

Just ignore the tokens tagged as "Collectibles" (if there are any) in the JSON. We still show all and only ERC721 and ERC1155 in the hardcoded Collectibles tab. Otherwise it wouldn't work.

Would be good to improve in https://unstoppabledomains.com/apps:

• [ ] Remove spurious space in "Alpha Wallet"
• [ ] Update logo
• [ ] Appear under "Featured"

i.e show this if ENS/custom name is not available:

Old version which shows "Wallet":

Don't change the title if ENS/custom name is available, so keep this:

Also keep the "Wallet" in the bottom tab bar.

@oa-s will send you a URL offline

1. Visit URL in desktop browser
2. Make sure Goerli is enabled
3. Click "Connect wallet"
4. Approve WalletConnect session
5. Click "Mint Test Devcon Souvenir Token"
6. WalletConnect actionsheet to sign transaction appears
7. Tap Confirm
8. Actionsheet closes

Expected

9. Transaction sent — a confirmation screen appears about transaction sent and show up as done or pending in Activity

Observed

9. Nothing seemed to have happened

Works if accessed in dapp browser

So we can support this: user doesn't have to do scan the QR code twice like this:

1. Have X enabled only
2. Scans WalletConnect QR code which ask for Y (or for WalletConnect v2, both X and Y)
3. Cancel prompt
4. Enable Y
5. Generate another QR code and scan again

Logic code has to be modified so that enabling/disabling chains on the fly (without UI) doesn't reload the entire app's UI. We can work on the UI especially for WalletConnect as a separate PR.

Works with #6034 so testnets can be enabled

Follow up:

• [ ] UI change in Settings
• [ ] UI change for WalletConnect v1 and v2
• [ ] UI change for other chain switching prompt (dapp browser, import token, what else?)

A. Remove the switch for mainnet B. Keep the switch for testnet — unchanged C. Keep the prompt for acknowledgment when enabling testnets D. List of mainnets should always be available in the list of chains E. The header in Wallet tab for the current testnet-mode should be removed F. We always show the header with fiat value for mainnets that are enabled (if any, otherwise $0), in Wallet tab