Feat/choose visible wallets

[RealGoodProgrammer]

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
adamant-im	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 7, 2024 6:52pm

[github-actions[bot]]

Deployed to https://msg-adamant-pr-570.surge.sh 🚀

[bludnic]

Deployed to https://msg-adamant-pr-570.surge.sh 🚀

Re-deployed surge. Now it works @RealGoodProgrammer

[bludnic]

I suggest to simplify names in the Wallets module

actions

setWalletSymbolsTemplates → updateWallets initWalletsSymbolsTemplates →initWallets

getters getAllOrderedWalletSymbols → wallets getVisibleOrderedWalletSymbols → visibleWallets

mutations

setWalletSymbolsTemplates → setWallets

[bludnic]

@RealGoodProgrammer Please merge dev and resolve conflicts. Check remaining review comments.

[adamant-al]

@RealGoodProgrammer

How much ADM does a user have on a screenshot? LSK? DOGE?

• Replace three dots ... with ellipsys (a single symbol) …
• Place … before the last digit, round it
• No … after . (whole and decimals divider)
• Show ~ before the number, if you show it not in full

As we discussed with @bludnic, we can round it up to 4 decimals. Otherwise, do the task UI: Balance decimals on Account tab (copied from iOS) https://trello.com/c/Yeh2AR7P fully.

[adamant-al]

@RealGoodProgrammer Wallet list is empty:

[bludnic]

Wallet list is empty:

@adamant-al It may be related to the new adamant-wallets structure. @RealGoodProgrammer please merge the dev branch into current

[RealGoodProgrammer]

@bludnic There is still no coins even after merge. Any ideas?

[bludnic]

Rolling back that commit seems solve the issue https://github.com/Adamant-im/adamant-im/pull/570/commits/f72995c1a62f303decbdcda8255279fe96f98b3e Look at the App.vue changes @RealGoodProgrammer

[adamant-al]

@RealGoodProgrammer Fix: Update ETH balance and gas price when any of ERC20 coins is active.

[bludnic]

@RealGoodProgrammer Merge the dev branch. There are conflicts.

[adamant-al]

@RealGoodProgrammer

• Change icon size from 36 to 32
• Align digits to the right, and a bit bottom
• For "Blockchain BTC" use regular font-weight, 400 ⟶ 300
• Ensure that in test account, Chrome ⟶ Mobile ⟶ iPhone SE shows correctly

[RealGoodProgrammer]

[adamant-al]

@RealGoodProgrammer Align both lines with numbers to bottom.

[adamant-al]

@bludnic @martiliones Please review.

[RealGoodProgrammer]

@adamant-al , all suggestions has been implemented

[github-actions[bot]]

Successfully tore down https://msg-adamant-pr-570.surge.sh 🥲