Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Wallet: NFT Details V2 #8064

Closed
4 tasks done
nuo-xu opened this issue Sep 14, 2023 · 1 comment · Fixed by #8408
Closed
4 tasks done

Wallet: NFT Details V2 #8064

nuo-xu opened this issue Sep 14, 2023 · 1 comment · Fixed by #8408
Assignees
@nuo-xu
Labels
Epic: Brave Wallet QA Pass - iPad QA Pass - iPhone X QA Pass - iPhone QA/Yes release-notes/include
Milestone
1.61

Comments

@nuo-xu
Copy link
Contributor

nuo-xu commented Sep 14, 2023 •
edited
Loading

Tasks
Preview Give feedback
@nuo-xu nuo-xu added this to Web3 Sep 14, 2023
@HEagle18 HEagle18 moved this to Backlog in Web3 Sep 15, 2023
@nuo-xu nuo-xu added the QA/Yes label Nov 3, 2023
@nuo-xu nuo-xu mentioned this issue Nov 14, 2023
Merged
11 tasks
@github-project-automation github-project-automation bot moved this from Backlog to Done in Web3 Nov 17, 2023
@iccub iccub added this to the 1.61 milestone Nov 29, 2023
@srirambv
Copy link
Contributor

srirambv commented Dec 5, 2023 •
edited
Loading

Verification passed on the following devices running 1.60 (23.11.30.17)

  • Verified steps from #8408
  • Verified NFT shows account blockie and network icon
  • Verified when NFT image is not loaded it shows a generic image
  • Verified opening an NFT shows the NFT details
  • Verified 3 dot menu on each NFT
  • Verified 3 dot menu on collected NFT shows Send, Hide, Don't Show in Wallet
  • Verified selecting Send pre-populates and loads the Send screen
  • Verified selecting Hide moves the NFT into Hidden section
  • Verified immediately selecting Hide and opening 3 dot menu shows Unhide
  • Verified selecting Don't Show in Wallet shows the remove from wallet message, selecting Remove removes it from wallet
  • Verified 3 dot menu on Hidden NFT shows Mark as Not Junk and Don't Show in Wallet
  • Verified selecting Mark as Not Junk removes the Junk label and moves the NFT into Collected list
  • Verified selecting Don't Show in Wallet shows the remove from wallet message, selecting Remove removes it from wallet
iPhone 13 (iOS 17.2) iPad (iOS 16.7.2) iPhone 7+ (iOS 15.8)
8064-iPhone.13.MP4
8064-iPad.mp4
8064-iPhone.7+.MP4

Note: iOS 15 verification blocked on #8513

@Uni-verse Uni-verse mentioned this issue Jan 23, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nuo-xu nuo-xu

Labels
Epic: Brave Wallet QA Pass - iPad QA Pass - iPhone X QA Pass - iPhone QA/Yes release-notes/include
Projects
Web3
Archived in project
Milestone
1.61
Development

Successfully merging a pull request may close this issue.

Fix 8064: Some v2 updates in NFT details brave/brave-ios
3 participants
@nuo-xu @iccub @srirambv