Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NFT button missing after app update via apk from site https://tonkeeper.com/ #28

Closed
isopen opened this issue Sep 18, 2022 · 7 comments
Closed
Labels
wontfix This will not be worked on

Comments

@isopen
Copy link

isopen commented Sep 18, 2022

Bug Type

UI/UX

Reproduction steps

Actual result

NFT button missing after update via apk.

Expected result

NFT button present after update via apk

Suggested Severity

Critical

Device

Desktop (please complete the following information):

  • OS: Android 12
  • Browser Android 12
  • Version 12
  • Tonkeeper Версия 2.6

Additional Context

No response

@olyaMay
Copy link
Contributor

olyaMay commented Sep 20, 2022

Hi. NFT button now displays only in cases when you have NFTs on your balance.

@olyaMay olyaMay closed this as completed Sep 20, 2022
@olyaMay olyaMay added the wontfix This will not be worked on label Sep 20, 2022
@isopen
Copy link
Author

isopen commented Sep 20, 2022

Hi. NFT button in Tonkeeper

@isopen
Copy link
Author

isopen commented Sep 20, 2022

NFT button not on the screen. This is a bug

@isopen
Copy link
Author

isopen commented Sep 20, 2022

Your condition doesn't always work. Because I have never had an NFT on my balance sheet, but I did have an NFT button.

@isopen
Copy link
Author

isopen commented Sep 20, 2022

There is no NFT on the balance, but the button has appeared (Magic button).
You can check the availability of NFT using the address EQADXaOqEmIzxvJBN9oQeaNYEA9Q5EBIYxQUYF7uM6InWJCr

@olyaMay Why was the report closed?

@MariaBit
Copy link
Contributor

Hello!

At the moment NFT button should be visible only when you have at least one NFT on your balance.
Because of that we closed this issue (#28).

However, you filed another bug #51 and it seems valid.
We are investigating it and will be back with an answer why NFT button became visible when there is no NFT.

Thanks!

@isopen
Copy link
Author

isopen commented Oct 11, 2022

The NFT button may not appear in one more case. When integration not available. Even if the user has NFT.

voloshinskii pushed a commit that referenced this issue Jun 24, 2024
voloshinskii added a commit that referenced this issue Jun 25, 2024
* feat(mobile): add signer & ledger support (#831)

* feat(mobile): signer support

* feat(mobile): ledger support (#835)

* bump(mobile): 4.5.0 (#836)

* fix(mobile): hidden amount background (#829)

* fix(mobile): fix ledger on android (#837)

* debug(mobile): add init error toast (#841)

* fix(mobile): reduce storage size (#842)

* feat(mobile): notcoin support (#843)

* feat(mobile): notcoin transfer action

* feat(mobile): burn notcoin vouchers

* NotcoinVerifyModal & fixes

* notcoin button in browser

* Update BurnVouchersModal.tsx

* fix ledger rename

* fix(mobile): signer & ledger flags (#844)

* feature(mobile): battery top-up (#833)

* wip

* fix ShowBalance button color

* wip

* feature(mobile): Battery top-up

* minor logic fix

* remove unused import

* persist keyboard

* fix(mobile): fix loadBatteryConfig

* fix condition

* fix keyboardAvoiding

* fix(mobile): remove replaceModal call

* Battery top-up improvements

* Misc battery updates

* add missing return

* fix(mobile): Battery topup QA fixes

* fix(mobile): Force relayer use for battery top-up (#845)

* fix(mobile): customize ledger wallet name (#846)

* fix(mobile): fix useShouldShowTokensButton (#847)

* fix(mobile): Update flag for recharge module (#848)

* fix(mobile): Update flag for recharge module

* fix: flag for promo list item

* fix(mobile): Topup QA fixes (#849)

* fix(mobile): Topup QA fixes

* fix merge bug

* fix(mobile): release fixes (#850)

* fix(mobile): show less charges when gift battery for friend (#851)

* fix(mobile): show less charges when gift battery for friend

* bump codeVersion

* fix(mobile): fix burn_not_available toast (#852)

* fix(mobile): fix TON position (#854)

* fix(mobile): stop support tonkeeper domain in manifest

* fix(mobile): Move TON to bottom

* fix(mobile): 4.5.0 qa fixes (#853)

* fix(mobile): burn voucher tx amount (#855)

* fix(mobile): Fix recharge asset logic (#856)

remove unnecessary logic

* feature(mobile): WELCOME TO FAMILY 🦫 (#857)

* fix(mobile): add signer_about_url to config (#858)

* fix(mobile): fix crash (#859)

* fix(mobile): fix crash

* cleanup

* fix(mobile): Remove useAnimatedKeyboard hook (#860)

* fix(mobile): Fix some texts (#861)

* fix(mobile): Fix some texts

* fix reserved amount

* fix(mobile): Remove subtitle

* fix (#862)

* bump 4.5.1 (#863)

* fix(mobile): Join TG step (#864)

* fix(mobile): Join TG step

* fix amount

* fix amount

* fix(mobile): bunch of fixes for hotfix (#865)

* fix(mobile): Adaptive wallet buttons (#868)

* fix(mobile): NFT trust field support (#869)

* fix(mobile): Small refactor createWalletInstance (#870)

* fix(mobile): Refactor createWalletInstance

* Some refactor

* bump(mobile): 4.5.3 (#871)

* fix(mobile): Invalidate unknown watch-only contracts properly (#872)

* fix(mobile): Fix breaking change (#873)

* fix(mobile): scroll on switch wallet (#875)

* fix(mobile): diamonds accent color (#876)

* feat(mobile): slider text animation (#867)

* feat(mobile): bump firebase (#877)

* feat(mobile): update ios icon (#878)

* fix(mobile): tonconnect fixes (#880)

* feature(mobile): Mark scam NFTs (#874)

* wip

* Add scam endpoint

* fix endpoint

* bump(mobile): 4.6.0

* Hide spam NFTs from history

* feature(mobile): Battery refunds dApp (#879)

* feature(mobile): Battery refunds dApp

* bump(mobile): 4.6.0

* fix(mobile): disable Navbar

* fix(mobile): refund && scam fixes (#883)

* fix(mobile): Button colors

* fix(mobile): Mark transactions with spam NFTs, update naming

* fix(mobile): fix for NFTs with nullable address (#885)

* bump(mobile): 4.6.1

* add testflight build details

* fix test details

* fix(mobile): Bunch of fixes for release

* feat(mobile): wallet W5 (#2)

* fix(mobile): Battery min amount

* feature(mobile): Gasless transactions

Bring back inscriptions support

wip

feature(mobile): Gasless payments

feature(mobile): Gasless payments

fix gasless payload

fundReceiver -> excessesAddress

fix excessesAccount

fix type

fix gasless

fix forward amount

get excessesAccount safely, fix NFT send

fix gasless without balance

fix(mobile): Gasless support from deeplink

Small fixes

* small fixes

* fix(mobile): w5 fixes (#7)

* fix(mobile): fix loadRate method (#8)

* bump(mobile): 3.6.1

* fix(mobile): Add TK_RELAYER_FEE opcode

* fix deeplinks and use recharge methods from cache

* fix sendAll

* bump to 4.6.1...

* fix sendMode for w5

* fix(mobile): optimize recursive fee estimation

* fix(mobile): memo

* fix(mobile): fix estimate fee when transfer by deeplink

* fix(mobile): Send fixes

* fix(mobile): fix jetton swap params

* fix fail

* feature(mobile): Buy USDT

* feature(mobile): Migrate from jUSDT

* fix

* remove console.log

* fix(mobile): Validate dApp URL

* fix(mobile): safe pending_transactions

* bump(mobile): 4.6.2

* fix double toNano call

* Bump(mobile): 4.6.2

* feature(mobile): support gasless BOLT

* fix param

* bump(mobile): 3.7.0

* bump(mobile): 4.7.0

* fix(mobile): design review fixes (#22)

* fix(mobile): update font type

* fix(mobile): Design review fixes

* Add "TON" tag for tether

* fix suggest for gasless

* fix(mobile): report not scam too

* fix(mobile): Top spacing on Android

* feature(mobile): More locales (#23)

* feature(mobile): More locales

* Android support

* fix(mobile): bump react-native-sse (#24)

* fix(mobile): Last-Event-ID header (#25)

* fix(mobile): fix exchange layout (#26)

* fix(mobile): signer fixes (#28)

* hotfix(mobile): 4.6.3 (#27)

* fix(mobile): Optimize requests

* bump(mobile): 4.6.3

* fix(mobile): Minor improvements (#29)

* fix(mobile): fix multiple pending events (#30)

* fix(mobile): fix multiple pending events

* fix iteratee

* fix(mobile): bunch of improvements (#31)

* fix(mobile): Bring back ErrorBoundary (#33)

* fix(mobile): Fix lockups (#32)

* fix(mobile): Fix lockups

* catch exceptions on emulation

* fix(mobile): Fix ErrorScreen (#34)

* fix(mobile): Fix condition (#35)

* fix(mobile): fix logout text position (#36)

* fix(mobile): release fixes (#37)

* fix(mobile): report as scam only if needed

* move backup status from prev wallet to w5

* fix gasless for signer

* fix(mobile): fix condition

* fix(mobile): bring back config load

* fix(mobile): force ltr in native code

* fix(mobile): fix android deeplinking handling

* fix(mobile): fix tr locale

* fix(mobile) show all button for sell (#38)

* fix(mobile): signer fixes (#39)

* fix(mobile): fix invalid encrypted comments (#40)

* fix(mobile): fix invalid encrypted comments

* feature(mobile): Refunds List.Item

* fix(mobile): bring back isActiveAccount check (#41)

* fix(mobile): Drop animation (#42)

* fix(mobile): Drop subscriptions and fix staking

* fix(mobile): Fix lockup send (#43)

* bump(mobile): 4.7.1

---------

Co-authored-by: Andrey Sorokin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants