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

ABW-1286 Thumbnails for NFTs etc #456

Merged
merged 31 commits into from
May 1, 2023
Merged

ABW-1286 Thumbnails for NFTs etc #456

merged 31 commits into from
May 1, 2023

Conversation

kugel3
Copy link
Contributor

@kugel3 kugel3 commented Apr 25, 2023

Jira ticket: paste link here

Description

Adds thumbnails for tokens, NFT resources and the NFT images themselves, in the account view.

Notes

More places to get images in the next PR.

Screenshots

With this PR, these screens get properly downloaded and handled image data:

PR submission checklist

  • I have tested account to account transfer flow and have confirmed that it works

@kugel3 kugel3 marked this pull request as draft April 25, 2023 11:46
@kugel3 kugel3 force-pushed the ABW-1286_Thumbnails branch 4 times, most recently from 756ca23 to f226286 Compare April 28, 2023 13:58
@kugel3 kugel3 changed the title Abw 1286 thumbnails ABW-1286 Thumbnails for NFTs etc Apr 28, 2023
@kugel3 kugel3 marked this pull request as ready for review April 28, 2023 14:37
Copy link
Contributor

@CyonAlexRDX CyonAlexRDX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Good job! Some minor changes proposed

Copy link
Contributor

@CyonAlexRDX CyonAlexRDX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kugel3 kugel3 merged commit 5b28f24 into main May 1, 2023
@kugel3 kugel3 deleted the ABW-1286_Thumbnails branch May 1, 2023 20:45
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants