- Unit tested helper function to get circulating supply
- Unit tested helper function to get (net) burn or mint
- Implement functions
- Clean up rendering (spinner instead of "Loading..." ?)
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Apr 3 2024
Apr 2 2024
Now if you enter in an invalid message (i.e. over length limit), then close the message toggle, the Send button is disabled even though theoretically it should be sending the tx without the message.
Is the following expected behavior?
- fill in valid address, amount and message
- close the message toggle indicating the user has changed their mind about sending this with a message
- click send and observe on the explorer that the message was sent anyway
Apr 1 2024
This is the same in master.
Mar 31 2024
Subject to some material rebases
subject to rebasing to latest changes to txHistory parsing and sendToken UI
Mar 30 2024
version bump and rebase
How do you feel about using a carousel type UI to browse through the list of tokens? It's something that NFT wallets in ethereum uses to great effect because the UI size is ringfenced regardless of how many tokens you have.
In D15810#357697, @emack wrote:
grep -r console.log src/ -- still a few left, but they are removed in D15804
It current enables the Send button when the aggregate send amount is exactly the same as the wallet balance, which shouldn't be the case because then there wouldn't be enough to pay for the txFee.
Mar 29 2024
The switches on the airdrop screen are too large relative to the rest of the UI, should resize to match the ones on config
Accepted, assuming the overall UI here will be improved separately.
In D15804#357425, @bytesofman wrote:In D15804#357421, @emack wrote:Testing in incognito solved the issue above, however now when creating a new token with an icon, getting the following:
The token is created regardless.
Based on my previous reviews of the token server I think this is due to the icon being over 500kb? The test png I used was 2mb. In which case the size error here should be clearer to the user so they can adjust it themselves.
Edit: nvm, tested it with a 114kb png and it's still getting rejected as well with the same message and console logs.
is this from localhost:3000 ? could be the whitelist at token server
will check it out
Testing in incognito solved the issue above, however now when creating a new token with an icon, getting the following:
Burning the remaining balance of an eToken results in a blank page rather than being routed back to the token list view (as per master/prod).
Same issue upon startup as D15804, however for reference D15748 loads fine without this issue.
Getting this uncaught exception upon startup. This is after npm ci.
Now consistently getting this uncaught exception upon the creation of a new token, line 593 of useWallet didn't seem to have an obvious cause for this.
Mar 28 2024
Mar 27 2024
Mar 26 2024
computer says no
Error workflow:
- Open Send and observe the camera auto opens as expected (since the config toggle is true)
- Toggle the auto open camera config to false
- Open Send and observe camera does not auto open
- Toggle the auto open camera config to true
- Open Send and the camera does not auto open again
- Despite refreshing app a few times, camera still does not auto open with the config set to true.
Mar 25 2024
Tested ok, just a minor nit in function comments
Mar 24 2024
It's not immediately clear whether each notification is an error, success or warning message at a quick glance. I would suggest using either a green tick/red cross/yellow exclamation mark to accompany each notification (via the icon prop) or customize the background color depending on the nature of the message. White text on black bg can be interpreted in different ways.
Mar 23 2024
Mar 22 2024
Much better
Error workflow:
- 100 XEC airdrop for 1c6c9c64d70b285befe733f175d0f384538576876bd280b10587df81279d3f5e, which should give you the following:
Mar 21 2024
I'm seeing a perma-lock on incognito mode (which has no existing wallet in storage).
- No errors in console.
- Already tried npm ci.
- Reloaded and hard-refreshed multiple times.
Likely the cashtabLoaded state var isn't being set in the onboarding process?
I'll ping you on tg to whitelist my IP. Given the changes it would be good if I can manually test the token creation process in case there's something not obvious to the int tests coverage.
Mar 19 2024
Is there a difference between an unknown token vs an invalid token that does not conform to SLPv1 or ALP standards?
Mar 16 2024
Updated prefix reservation
If you toggle Send Confirmations in config to true, then send a normal XEC tx, that setting resets to false after tx broadcast.
Mar 15 2024
Edge case: Send a token tx where you're sending the entire remaining token balance so that it needs to automatically route out of the SendToken.js and back to Home. The following bad state use warning is shown:
Mar 13 2024
So my steps were:
- Deploy v2.0.10 to netlify
- Create new wallet on there
- Bump version to v2.0.11, then redeploy to netlify
- Waited around with the wallet open but no modal.
- Closed and re-opened browser, upgrade modal shows up on load
- Click OK, cashtab refreshes but Configure.js is still showing v2.0.10
- Hard refresh browser, cashtab now shows v2.0.11