Skip to content
This repository was archived by the owner on Oct 4, 2023. It is now read-only.

[C-462] Fix legacy image loading for native#1300

Merged
dylanjeffers merged 1 commit into
mainfrom
dj-c-462-fix-legacy-profile-picture
May 10, 2022
Merged

[C-462] Fix legacy image loading for native#1300
dylanjeffers merged 1 commit into
mainfrom
dj-c-462-fix-legacy-profile-picture

Conversation

@dylanjeffers

Copy link
Copy Markdown
Contributor

Description

Fixes case where legacy images don't load properly in native.

  • Cleans up redundant extra fetch (libs already handles this logic)

How Has This Been Tested?

Legacy images load on native, and still load on desktop/mobile web

@audius-infra

Copy link
Copy Markdown
Collaborator

@raymondjacobson raymondjacobson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Big simplification 👍
I assume you tested the audius profile on web too right?

@dylanjeffers dylanjeffers merged commit 870c176 into main May 10, 2022
@dylanjeffers dylanjeffers deleted the dj-c-462-fix-legacy-profile-picture branch May 10, 2022 02:43
@sliptype sliptype mentioned this pull request May 13, 2022
@AudiusProject AudiusProject deleted a comment from linear Bot Sep 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants