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

[C-1427] Fix multiple issues with Collectibles on mobile #2296

Merged
merged 5 commits into from
Nov 23, 2022

Conversation

sliptype
Copy link
Contributor

Description

  • Remove control characters in nft urls
  • Support svgs (Image doesn't support svgs)
  • Respect collectible order
  • Add spinner

Dragons

Is there anything the reviewer should be on the lookout for? Are there any dangerous changes?

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.

How will this change be monitored?

For features that are critical or could fail silently please describe the monitoring/alerting being added.

Feature Flags

Are all new features properly feature flagged? Describe added feature flags.

@sliptype sliptype requested review from a team and Kyle-Shanks and removed request for a team November 23, 2022 02:17
@sliptype sliptype requested review from dylanjeffers and a team and removed request for Kyle-Shanks November 23, 2022 02:17
// Remove control characters from url
const sanitizedURL = item.url.replace(
// eslint-disable-next-line
/[\u0000-\u001F\u007F-\u009F]/g,
Copy link
Member

Choose a reason for hiding this comment

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

OOF

@sliptype sliptype merged commit 8eba94e into main Nov 23, 2022
@sliptype sliptype deleted the sk-c-1427-collectibles branch November 23, 2022 02:36
audius-infra pushed a commit that referenced this pull request Nov 26, 2022
[1264e58] Bump ios version to 1.1.41, lint (#2308) Dylan Jeffers
[8e88d8d] [C-1549] Fix mobile share-drawer (#2307) Dylan Jeffers
[4acd94b] Bump ios (#2306) Sebastian Klingler
[f489853] [C-1326] Fix open in app drawer to open the app properly (#2286) Kyle Shanks
[1db362c] [C-1531] Remove open in app from landing page (#2304) Raymond Jacobson
[67196b8] iOS fast follows (#2305) Sebastian Klingler
[fed67d3] Fix edit images (#2302) Sebastian Klingler
[2d17c4a] Bump android image size (#2301) Sebastian Klingler
[b16413f] Can be faster (#2300) Raymond Jacobson
[0ea02e5] Fix types (#2299) Sebastian Klingler
[f02ad81] [C-1411] Fix reachability issues with new splash screen (#2298) Raymond Jacobson
[9afd61c] Use navigation.navigate for search (#2297) Sebastian Klingler
[996a451] [C-1404] Add initial mobile wallet-connect (#2294) Dylan Jeffers
[8eba94e] [C-1427] Fix multiple issues with Collectibles on mobile (#2296) Sebastian Klingler
[d776787] Improve mobile image fetching (#2295) Sebastian Klingler
[d3eddcc] [C-1535] Add mobile-upload analytics, fix remix field (#2293) Dylan Jeffers
[ab6ece8] [C-1411] Improve loading splash (#2289) Raymond Jacobson
[9d9ac84] [C-1328] Add edit/delete track mobile overflow options (#2292) Dylan Jeffers
[82aec6c] [C-1509] Add input-accessory-view (#2291) Dylan Jeffers
[916ada5] Fix premium content types (#2290) Sebastian Klingler
@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.

2 participants