[C-3633] Fix upload tags #7246
Merged
[C-3633] Fix upload tags #7246
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
|
Preview this change https://demo.audius.co/dj-c-3633-fix-upload-tags |
amendelsohn
approved these changes
Jan 19, 2024
audius-infra
pushed a commit
that referenced
this pull request
Jan 20, 2024
[df201c4] [C-3651] Add box-shadow to text-input (#7259) Dylan Jeffers [3436391] Remove track player patch (#7256) Dylan Jeffers [2dd727a] [C-3633] Fix upload tags (#7246) Dylan Jeffers [336abe5] [PAY-2362] Allow offline download (#7253) Saliou Diallo [bdc4997] Re-add download buttons to mobile track page (#7254) Reed [5f2045c] Add auto upgrade logs to vector (#7251) Isaac Solo [d87e13f] Update styles to wrap properly (#7248) Saliou Diallo [1599367] [PROTO-1598] Use tRPC in ddex (#7150) Theo Ilie [09b5dc7] Qm sync (#7250) Steve Perkins [cc892ea] Remove scheduled release filter from sitemaps (#7247) Isaac Solo [3211ca2] Fix stylelint (#7245) Saliou Diallo [270f107] Increase chat reaction emoji sizes (#7241) Reed [26b0f0c] [PAY-2331] Add lossless download info section in upload (#7219) Saliou Diallo [2a15a64] v1.5.62 (#7243) Dylan Jeffers [b94153b] Convert mobile DownloadRow to harmony (#7237) Reed [62b8611] [C-3597] Add ellipses to track tile titles (#7234) Dylan Jeffers [eba9299] [C-3678, C-3657] Fix android radial-gradient bug, Fix sign-up panel top border-radius (#7236) Dylan Jeffers [bf9d2ea] Fix sign ups failing halfway through saga (#7240) JD Francis [f355286] Re-add downloads when lossless downloads flag disabled (#7235) Reed [0ffd417] [C-3677] Fix onclick for upload a track chip (#7232) Andrew Mendelsohn [f9132ca] Filter sitemaps properly (#7233) Raymond Jacobson [5f8d5d2] Fix max width issue on select artists page (#7229) JD Francis [dc4fa10] [DOCS] - repair reference links and set config to throw errors on breaks (#7231) Sam Gutentag [f6ad76f] [PAY-2361] Fix search es with new gated fields (#7230) Saliou Diallo [5d97832] [PAY-2319] Add offramp on web (#7153) Randy Schott [eab69ec] [PAY-2364] Fix nft gated tracks (#7228) Saliou Diallo [12af10e] [C-3637] Add OTP flow for new sign-up (#7227) Dylan Jeffers [4d68c2a] [C-3642] Handle "user past account creation" edge case scenarios in web (#7225) JD Francis [60b843d] Fix DDEX dockerfile (#7226) Michelle Brier [c3c2148] [Web] [Libs] Disconnect dashboard wallet - web and SDK changes (#7209) nicoback2 [b7d03e3] [C-3640] Reduce toast duration app-wide (#7224) Andrew Mendelsohn [120c8f6] [C-3639] Fix empty album tab state (#7222) Andrew Mendelsohn [059312d] [C-3621] Fix scrolling issues on new sign up Password & Handle pages (web) (#7204) JD Francis [557fd96] Get public IP in nginx (#7223) Isaac Solo [0046ab4] [C-3149 C-3122 C-3051 C-3625 C-3481] Fix add track to playlist (#7221) Dylan Jeffers [ba711b6] [C-3583 C-3634 C-3404] Add harmony-native to native sign up (#7210) Dylan Jeffers [335cfa4] Remove yield* in JS file (#7134) Marcus Pasell [fa7a9d9] Make FilterButton a controlled component (#6993) Marcus Pasell [1eb3aa1] [C-3649] Update create password screen to move when keyboard opens to display validation (#7220) Kyle Shanks [55b5192] [PAY-2356] Mobile track screen stems & download section (#7207) Reed [c0b2b66] [Web] Add OTP to OAuth login flow [C-3643] (#7212) nicoback2 [f5cee04] Openresty container (#7214) Steve Perkins [33f7597] Export plain button from harmony native (#7217) Kyle Shanks [13412f3] CircleCI self-hosted runners: lower cleanup thresholds (#7216) Danny [c57618e] Fix otp after #7196 (#7215) Raymond Jacobson [31629e6] [C-3638] Fix saga ci (#7211) Dylan Jeffers [ab09783] Fix ISO date format for dayjs (#7206) Isaac Solo [abc71bf] Fix stems & downloads appearing erroneously web (#7202) Reed [fb38737] [PAY-2351] Fix height on add funds (#7172) Raymond Jacobson [38221f2] [C-3614, C-3615] Update image storage for new sign up flow to conform with old structure (#7200) Kyle Shanks [b0bf87d] use slog-gorm package (#7199) Steve Perkins [ce50f32] Gitignore snippets (#7201) Reed [204894c] Fix audius-cmd test (#7208) Saliou Diallo [f5226c7] [C-3630] Refactor signon screens to use native-harmony (#7192) Dylan Jeffers [bcc180f] Bypass OTP for testflight account (#7196) Saliou Diallo [94ec3d0] Update mobile version to 1.1.83 for release (#7203) Dylan Jeffers [ab1d3e9] Add USDC gated downloads flow (#6899) Saliou Diallo [966e438] [C-3608] Increase pressable area for cover photo upload (native) (#7165) JD Francis [9012df5] [C-2190] Fix track player interruptions (#7198) Dylan Jeffers [d12d824] [C-3632] Return null instead of NOT_FOUND in trpc-server (#7184) Andrew Mendelsohn [b391ada] Bump version to 0.6.19 audius-infra [ecbd9b1] Bump version to 0.6.18 audius-infra [625839b] Fix repair signup (#7197) Marcus Pasell [fc5131e] Add sendgrid param (#7195) Raymond Jacobson [d530b0e] Bump app builds (#7194) Raymond Jacobson [8d031a4] [PROTO-1599] Add OTP support (#7193) Raymond Jacobson
schottra
added a commit
that referenced
this pull request
Jan 22, 2024
…xing * origin/main: (37 commits) [DVRL-3] optimize image assets (#7238) PROTO-1607: healthz rm GA dependency (#7264) [C-3696] Fix lineups and recommendations (#7263) Revert "[C-3510] Move mobile env from system config to ts env (#6935)" (#7268) Make ddex tRPC Dockerfile run (#7261) Attempt to circumvent docker-compose regression (#7266) Bump version to 0.6.20 qm sync: handle non-200 status, more tests. (#7265) [PAY-2367] Add migration to repair tips (#7252) [C-3510] Move mobile env from system config to ts env (#6935) v1.5.63 [C-3651] Add box-shadow to text-input (#7259) Remove track player patch (#7256) [C-3633] Fix upload tags (#7246) [PAY-2362] Allow offline download (#7253) Re-add download buttons to mobile track page (#7254) Add auto upgrade logs to vector (#7251) Update styles to wrap properly (#7248) [PROTO-1598] Use tRPC in ddex (#7150) Qm sync (#7250) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Fixes issue where album tags were incorrectly merging with track override tabs, leading to cases where tracks can have more than 10 tags.
Note we are checking that the track tags are explicitly null, ie not overridden, there is a world where user clears all ablum tags from the track, which we should preserve