Uh oh!
There was an error while loading. Please reload this page.
[PE-6660] Add new send tokens modal flow - #12777
Conversation
|
🚨 Bugbot Trial ExpiredYour Bugbot trial has expired. Please purchase a license in the Cursor dashboard to continue using Bugbot. |
| // Send the tokens using the audius backend | ||
| // This method throws errors instead of returning them | ||
| await audiusBackend.sendWAudioTokens({ |
There was a problem hiding this comment.
so we don't have a send in sdk yet?
There was a problem hiding this comment.
i think it technically uses sdk but is a weird wrapper function
| } catch (error) { | ||
| console.error('Error sending tokens:', error) | ||
| // Use getErrorMessage to extract the error message |
| : BigInt(0) | ||
| const formatAmount = (amount: bigint) => { | ||
| return new FixedDecimal(amount, tokenInfo?.decimals).toLocaleString( |
| ) | ||
| } | ||
| const handleCheckboxChange = (event: React.ChangeEvent<HTMLInputElement>) => { |
There was a problem hiding this comment.
nit: maybe just import ChangeEvent directly
| return ( | ||
| <Flex direction='column' gap='xl' p='xl' alignItems='center'> | ||
| <Text variant='body' size='l' color='subdued'> | ||
| Loading token information... |
| amount={formatBalance(currentBalance)} | ||
| /> | ||
| <Divider orientation='horizontal' color='default' /> |
| } | ||
| return ( | ||
| <Flex direction='column' gap='xl' p='xl'> |
There was a problem hiding this comment.
column instead of direction='column'
| color='default' | ||
| css={{ wordBreak: 'break-all' }} | ||
| > | ||
| -{formatAmount(amount)} {tokenInfo.symbol} |
There was a problem hiding this comment.
maybe put all of this into a message fn
| const getAmountDescription = () => { | ||
| return messages.amountDescription.replace( | ||
| '{symbol}', | ||
| tokenInfo?.symbol || 'tokens' |
There was a problem hiding this comment.
ai prefers || but our style guide prefers ??
| const handleConfirm = async () => { | ||
| setState((prev) => ({ ...prev, step: 'progress' })) | ||
| setError('') // Clear any previous errors |
| }) | ||
| // If successful, move to success step | ||
| setState((prev) => ({ ...prev, step: 'success' })) |
There was a problem hiding this comment.
is this something we could do in the hook itself? or is that too web-specific?
There was a problem hiding this comment.
ohh good q, ill consider it. my guess is that we want to do a different pattern for mobile since itll be screen based.
| dismissOnClickOutside={state.step === 'input'} | ||
| showDismissButton={state.step === 'input'} | ||
| > | ||
| {state.step === 'input' && ( |
| css={{ alignSelf: 'flex-start' }} | ||
| onClick={() => { | ||
| window.open( | ||
| `https://explorer.solana.com/address/${destinationAddress}`, |
There was a problem hiding this comment.
maybe this should be in common somewhere
| initialDestinationAddress?: string | ||
| } | ||
| export interface SendTokensConfirmationProps { |
There was a problem hiding this comment.
why are these interfaces instead of types
| } | ||
| const BalanceSectionContent = ({ mint }: AssetDetailProps) => { | ||
| const BalanceSectionContent = ({ mint, onSend }: AssetDetailProps) => { |
dharit-tan
left a comment
There was a problem hiding this comment.
some cleanup needed but stamping for now
Uh oh!
There was an error while loading. Please reload this page.
[b7a7478] [PE-6729|PE-6728|PE-6724] Various minor artist coin tweaks (#12782) JD Francis [48903d3] [PE-6744] Connect ExternalWalletsCard to ExternalWalletsScreen on mobile (#12793) KJ [8dff4f9] OEmbed: Don't use OG url (#12792) Marcus Pasell [76aae83] [QA-2290] Mobile USDC purchase recovery (#12791) Reed [e99bbc8] Fix buy-audio wrong change formatting (#12787) Reed [9d33032] $AUDIO balance includes staked (#12790) Reed [a15f37e] Bump version of Jupiter price API (#12789) Farid Salau [8cdd634] [PE-6717] Update default buy/sell token pair based on path (#12788) KJ [b1792fb] [PE-6721] Fix badge hover card in left nav profile (#12784) Reed [e656e9f] Update user bank deriver to use artist coins endpoint (#12785) Farid Salau [52f3c35] Allow direct DBC swaps (#12776) Farid Salau [2e341b2] Remove hardcoded decimals values from Swap flow (#12763) Farid Salau [5e4a021] [PE-6735, PE-6738, PE-6739] Updates to the mobile coins screens (#12783) KJ [930bbe8] [QA-2182] Fix selected tip reaction (#12703) Dylan Jeffers [80085fe] [QA-1076] Fix notification drawer top inset (#12704) Dylan Jeffers [c93ad60] [PE-6660] Add new send tokens modal flow (#12777) Dylan Jeffers [571711c] [PE-6732, PE-6733, PE-6734, PE-6517] Add mobile AllCoinsScreen and CoinDetailsScreen (#12775) KJ [61aed02] Drop track tile on upload complete screen (#12780) Dylan Jeffers [881cd6c] [PE-6725] Remove ai-attribution from uploadTrack e2e test (#12779) Reed [b799ee3] Fix types and type safety in useConnectedWallets, useAudioBalance, useWalletAudioBalance, and useWalletAudioBalances (#12683) Marcus Pasell [629d200] [PE-6714] Only show add cash when no USDC balance (#12774) Reed [041f71e] [PE-6670] Redo discord flow with OAuth (#12772) JD Francis [22e7d18] [PE-6582] Fix mobile musical key filter (#12768) Dylan Jeffers [8af0b47] [PE-6726] Mobile Search QA (#12767) Dylan Jeffers [e2661ca] [PE-6709|PE-6710|PE-6707] Artist coin QA pass (#12761) JD Francis [935c054] update b challenge (#12764) Randy Schott [e32b36b] [PE-6676] Improve mobile explore content performance (#12739) Dylan Jeffers [246a276] [PE-6632] Fix selected category display for mobile web search/explore (#12737) Randy Schott [d155e23] Fix receive modal spacing (#12759) Reed
Description
Adds SendTokensModal
Adds useSendTokens
TODO: