Same NFT displayed multiple times - NFT dashboard #84
Comments
|
The issue still persists after having pulled the latest develop @Imamah-Zafar
|
7 tasks
this will get fixed when PR gets merged |
|
@Imamah-Zafar After pulling the latest develop, this bug isn't fixed. https://github.com/secretkeylabs/xverse-web-extension/pull/98/files
|
teebszet
added a commit
that referenced
this issue
Mar 14, 2024
* feat: init structure for send rune flow with flag for testnet only * add hook * add runes api client * feat: handle rune token display on home and token dashboard (#81) * wip * chore: tweak token dashboard for runes and add tokens to home send bottomsheet * remove TODOs that are handled now * touchup * chore: update text to support runes in receive screen * chore: dry * chore: remove runes list from client store * chore: remvoe todos related to texts * chore: remove useGetRuneFungibleTokens config todo --------- Co-authored-by: fede erbes <[email protected]> * Manage tokens screen for runes support (#84) * refactor key variables to improve readability * updates * major refactor stx coinslist & support runes in manage token page for testnet * remove conditonals and improve performance * add figma for mainnet coming soon * chore: add protocol icons for stacks, brc20 and runes tokens (#83) * chore: add protocol icons for stacks, brc20 and runes tokens * fix: token image loader and clean render logic * chore: use our runes logo and remove square styles * chore: use a default size const * fix: font size and add missing protocol to brc20 tokens * updates * fix stx icon issue --------- Co-authored-by: Terence Ng <[email protected]> * standardize currency typing and improve variable namings * revert logic + touchup * touchup * add rune icon * add typing * rename variable * progress * progress * fix button disabled * fix amount validations and display error msg * chore: use core pr version with runes support * Tim/eng 3814 receive runes screen design check (#91) * fix: receive bottom sheet icons and blur * Update en.json Co-authored-by: Terence Ng <[email protected]> --------- Co-authored-by: Terence Ng <[email protected]> * nits * Terence/fix edge case renderings (#95) * fix (#94) * fix * Update src/app/ui-library/input.tsx Co-authored-by: Tim Man <[email protected]> * fix * use variant * use brc * fix expression readability * fix sip coins which are hidden by manage token list should not be sendable in send coins list (#96) * update key * fix: remove unused route * fix: coin dashboard should show names and no protocol if none * standardize ticker icon for no image tickers * remove deprecated usage * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * fix up logic * refactor: manage tokens for runes (#98) * refactor: manage tokens for runes * fix: be more strict on toggle handler * chore: minor cleanup * chore: minor diff clean up * update remaining FT to sip-10 * testing (#93) * touchup currencyTypes logic to go back to using FT, fixes, prepare improved receive screen for STX, BTC, ORD * update receive coin screen to handle STX & BTC * fix type issue * revert console log * revert changes * logic fix * remove unused component * Tim/refactor manage tokens (#99) * refactor: brc20 and sip10 tokens should use react-query hooks and no longer store balances in redux * fix: define a redux migration callback to keep user settings for manage tokens * fix: brc20 ft list merge * update remaining FT to sip-10 * testing (#93) * touchup currencyTypes logic to go back to using FT, fixes, prepare improved receive screen for STX, BTC, ORD * update receive coin screen to handle STX & BTC * fix type issue * refactor: brc20 and sip10 tokens should use react-query hooks and no longer store balances in redux * test * revert * fix: also remove coins from redux state in migration * chore: consistent naming --------- Co-authored-by: Terence Ng <[email protected]> Co-authored-by: Terence Ng <[email protected]> * fix: fix typing and null case for redux migration function * chore: bump to latest beta core * refactorings * typo * use bigint and fix amount precision * fix debounce * update core * remove script info * chore: bump core version * hack: update confirmTransaction to handle runes v1 * allow display input/output * revert runes icon to use square * chore: core version with undefined runes fix * additional check * update logo * Empty rune list message scenario and error message (#107) * feat: init structure for send rune flow with flag for testnet only * add hook * add runes api client * feat: handle rune token display on home and token dashboard (#81) * wip * chore: tweak token dashboard for runes and add tokens to home send bottomsheet * remove TODOs that are handled now * touchup * chore: update text to support runes in receive screen * chore: dry * chore: remove runes list from client store * chore: remvoe todos related to texts * chore: remove useGetRuneFungibleTokens config todo --------- Co-authored-by: fede erbes <[email protected]> * Manage tokens screen for runes support (#84) * refactor key variables to improve readability * updates * major refactor stx coinslist & support runes in manage token page for testnet * remove conditonals and improve performance * add figma for mainnet coming soon * chore: add protocol icons for stacks, brc20 and runes tokens (#83) * chore: add protocol icons for stacks, brc20 and runes tokens * fix: token image loader and clean render logic * chore: use our runes logo and remove square styles * chore: use a default size const * fix: font size and add missing protocol to brc20 tokens * updates * fix stx icon issue --------- Co-authored-by: Terence Ng <[email protected]> * standardize currency typing and improve variable namings * revert logic + touchup * touchup * add rune icon * add typing * rename variable * progress * progress * chore: use core pr version with runes support * fix button disabled * fix amount validations and display error msg * Tim/eng 3814 receive runes screen design check (#91) * fix: receive bottom sheet icons and blur * Update en.json Co-authored-by: Terence Ng <[email protected]> --------- Co-authored-by: Terence Ng <[email protected]> * nits * fix (#94) * Terence/fix edge case renderings (#95) * fix (#94) * fix * Update src/app/ui-library/input.tsx Co-authored-by: Tim Man <[email protected]> * fix * use variant * use brc * fix expression readability * fix sip coins which are hidden by manage token list should not be sendable in send coins list (#96) * update key * fix: remove unused route * fix: coin dashboard should show names and no protocol if none * standardize ticker icon for no image tickers * remove deprecated usage * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * Update src/locales/en.json * fix up logic * refactor: manage tokens for runes (#98) * refactor: manage tokens for runes * fix: be more strict on toggle handler * chore: minor cleanup * chore: minor diff clean up * update remaining FT to sip-10 * testing (#93) * touchup currencyTypes logic to go back to using FT, fixes, prepare improved receive screen for STX, BTC, ORD * update receive coin screen to handle STX & BTC * fix type issue * revert console log * revert changes * logic fix * remove unused component * Tim/refactor manage tokens (#99) * refactor: brc20 and sip10 tokens should use react-query hooks and no longer store balances in redux * fix: define a redux migration callback to keep user settings for manage tokens * fix: brc20 ft list merge * update remaining FT to sip-10 * testing (#93) * touchup currencyTypes logic to go back to using FT, fixes, prepare improved receive screen for STX, BTC, ORD * update receive coin screen to handle STX & BTC * fix type issue * refactor: brc20 and sip10 tokens should use react-query hooks and no longer store balances in redux * test * revert * fix: also remove coins from redux state in migration * chore: consistent naming --------- Co-authored-by: Terence Ng <[email protected]> Co-authored-by: Terence Ng <[email protected]> * fix: fix typing and null case for redux migration function * chore: bump to latest beta core * refactorings * typo * use bigint and fix amount precision * fix debounce * update core * remove script info * chore: bump core version * hack: update confirmTransaction to handle runes v1 * allow display input/output * revert runes icon to use square * chore: core version with undefined runes fix * additional check * update logo * push * push --------- Co-authored-by: Tim Man <[email protected]> Co-authored-by: fede erbes <[email protected]> Co-authored-by: Victor Kirov <[email protected]> * update label * fix: overflow on send rune amound input, when long balance (#105) * fix: overflow on send rune amound input, when long balance * refactor: move the complications to the input component file * fix: set allowNegative=false to fix warnings for number format (#106) * fix: set allowNegative=false to fix warnings for number format * fix: styling of stx tx transaction history * fix balance becoming exponential string for large digits * fix large amount / decimal formatting * touchup * adjust gap to cater max scenario * update display ellipsis for high digit/decimal values * small fix * fix: css balance token tile * fix: cleaner css --------- Co-authored-by: Terence Ng <[email protected]> * fix (#109) * [ENG-3193] refactor: Refactor away from selectedOrdinal (#791) * commit changes * remove selectedOrdinal for raresat screen * update recover screens * fix translation text * revert non null assertion * fix bug * fix merge * fix (#109) * fixes (#110) * fixes * remove log * disable rbf * pull latest core to support rbf * Update core and fix map key for txn history --------- Co-authored-by: Victor Kirov <[email protected]> * [ENG-3193] refactor: Refactor away from selectedOrdinal (#791) * commit changes * remove selectedOrdinal for raresat screen * update recover screens * fix translation text * revert non null assertion * fix bug * fix merge * chore: bump core to v11.1.1 (#806) * chore: Bump core to 11.1.2 (#807) * chore: bump to latest beta core * update lock * use mobile values * fix render * Solve missing name * Fallback for SIP-10 FungibleTokens with missing name * chore: bump core to v11.3.0 --------- Co-authored-by: Terence Ng <[email protected]> Co-authored-by: Terence Ng <[email protected]> Co-authored-by: fede erbes <[email protected]> Co-authored-by: Victor Kirov <[email protected]> Co-authored-by: jordankzf <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component
NFT - DashboardNFT collection: https://gamma.io/collections/hallow-heads
https://user-images.githubusercontent.com/55587184/203743894-6e6d8e79-e230-4f13-8442-9ed9e5a371d2.mp4
Environment: latest develop
OS: Windows and Mac
Browser: Chrome Version 107.0.5304.107 (Official Build) (64-bit)
Severity
Major (Unable to proceed on a journey)Summary
Same NFT displayed multiple times.All the NFT's are from the same collectionhttps://gamma.io/SPR7RRXN8ZRXPNAFKPN2TB84J7T1RMZRGWQQV7P3Reproduce steps
Expected result
Unique NFT displayed.https://gamma.io/SPR7RRXN8ZRXPNAFKPN2TB84J7T1RMZRGWQQV7P3Actual result
Same NFT displayed .Remark
(Optional)The text was updated successfully, but these errors were encountered: