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

[PAY-2332] Add new stems and downloads upload UI #7333

Merged
merged 18 commits into from
Jan 31, 2024
Merged

Conversation

sddioulde
Copy link
Contributor

@sddioulde sddioulde commented Jan 25, 2024

Description

First PR to add new lossless / stems and downloads UI. Commits show the breakdown of the work.

#7382 merges into this one.

How Has This Been Tested?

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

local web dapp vs stage

@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/sd-stems-and-downloads

3 similar comments
@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/sd-stems-and-downloads

@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/sd-stems-and-downloads

@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/sd-stems-and-downloads

@@ -65,6 +74,12 @@
transform: scale(0.95);
}

.iconUpload.truncated {
margin: 0 12px;
width: 20px;
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: unit vars

@sddioulde sddioulde force-pushed the sd-stems-and-downloads branch from 67bbf7a to b20f102 Compare January 30, 2024 18:48
@audius-infra
Copy link
Collaborator

Preview this change https://demo.audius.co/sd-stems-and-downloads

) : null}
<Text>
{message}
<TextLink css={{ color: '#a30cb3' }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

this feels weird.. is this not in our design system?

}

export const TrackPreviewNew = (props: TrackPreviewProps) => {
const { isEnabled: isLosslessDownloadsEnabled } = useFeatureFlag(
const isLosslessDownloadsEnabled = getFeatureEnabled(
Copy link
Contributor

Choose a reason for hiding this comment

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

i think we prefer useFeatureFlag

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah was wondering. thx

{
key: DownloadTrackAvailabilityType.USDC_PURCHASE,
text: messages.premium,
icon: <IconCart size='s' fill='#858199' />
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto why do we need custom color

const [{ value }, , { setValue: setDownloadPrice }] =
useField<number>(DOWNLOAD_PRICE)
const [humanizedValue, setHumanizedValue] = useState(
value ? decimalIntegerToHumanReadable(value) : null
Copy link
Contributor

Choose a reason for hiding this comment

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

wonder if marcus's @audius/fixed-decimal could be useful here.

@@ -3,6 +3,7 @@
display: flex;
flex-direction: column;
overflow: auto;
gap: 24px;
Copy link
Contributor

Choose a reason for hiding this comment

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

unit var

</Flex>
</>
)
// return (
Copy link
Contributor

Choose a reason for hiding this comment

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

comment

@@ -142,10 +142,12 @@ export const FilterButton = forwardRef<HTMLButtonElement, FilterButtonProps>(
const handleButtonClick = useCallback(() => {
if (variant === 'fillContainer' && selection !== null) {
setSelection(null)
// @ts-ignore
Copy link
Contributor

Choose a reason for hiding this comment

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

why ts-ignore here?

@@ -83,6 +84,7 @@ export const TrackPreviewNew = (props: TrackPreviewProps) => {
} = props

const Icon = fileTypeIcon(fileType)
const iconStyle = isStem ? { width: 24, height: 24 } : undefined
Copy link
Contributor

Choose a reason for hiding this comment

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

spacing

lowerCaseFilename.includes(category.toString().toLowerCase())
) ?? StemCategory.OTHER
)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@@ -47,8 +50,9 @@
}

.dropzoneIcon {
height: 32px;
width: 32px;
margin: 24px;
Copy link
Contributor

Choose a reason for hiding this comment

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

unit var

}: USDCPurchaseRemoteConfig) =>
z
.object({
// [ALLOW_DOWNLOAD]: z.boolean(),
Copy link
Contributor

Choose a reason for hiding this comment

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

comment

Copy link
Contributor

@dharit-tan dharit-tan left a comment

Choose a reason for hiding this comment

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

mostly nitty stuff that can be addressed later but logic seems good to go.
mobile lint failure is my fault, now fixed in main.

… flows (#7382)

Co-authored-by: Saliou Diallo <saliou@audius.co>
@sddioulde sddioulde merged commit 513b141 into main Jan 31, 2024
6 of 7 checks passed
@sddioulde sddioulde deleted the sd-stems-and-downloads branch January 31, 2024 21:10
schottra added a commit that referenced this pull request Feb 1, 2024
…ramp-transaction

* origin/main: (26 commits)
  Change index_user_bank log to info instead of error (#7427)
  [PAY-2445] Separate email notifications for manual transfer and withdrawal (#7424)
  Add orig_filename to track/stems endpoint (#7426)
  Bump version to 0.6.28
  track page sagas to typescript (#7418)
  Fix duplicate unregistered SPs on healthz overview (#7406)
  Convert signon page sagas to ts (#7415)
  User sagas to typescript (#7405)
  Fix native "MultiselectRemove" icons (#7413)
  Deprecate all non-harmony Avatar in native (#7404)
  Fix profile lineup loading (#7409)
  Small fixes for lossless upload flow (#7412)
  Fix @audius/common auto-imports (#7414)
  Upload includes download_conditions splits, orig_filename for stems (#7411)
  Unbundle @audius/common (#7379)
  Refactor to use Harmony PasswordInput in native (#7410)
  Add DDEX publisher CI workflow (#7408)
  Fix isrc/iswc regex (#7407)
  [PAY-2332] Add new stems and downloads upload UI (#7333)
  [DVRL-2] Clean out docusaurus boilerplate content (#7239)
  ...
audius-infra pushed a commit that referenced this pull request Feb 3, 2024
[5646b21] Update dapp-store build artifacts audius-infra
[b194d17] [INF-652] Fix SSR Deploy (#7450) Sebastian Klingler
[03707c9] Upgrade dapp-store cli (#7420) Dylan Jeffers
[5a33f62] Fix copy icon color (#7441) Raymond Jacobson
[1fe48f9] Remove common command from root package.json (#7445) Sebastian Klingler
[c71c04d] Openresty cleanup (#7444) Steve Perkins
[080548b] Remove old cache track sagas js file (#7439) Kyle Shanks
[95060d1] Add bs58 lock changes (#7432) Saliou Diallo
[a0abc3a] Add stem orig_filename to tracks/<id>/stems (#7428) Reed
[b2e26bf] [PAY-2425] Fix DownloadRow text styles (#7438) Reed
[4759085] Openresty cleanup (#7423) Steve Perkins
[11537a8] Fix drawer icon sizing issue (#7417) JD Francis
[498e1c6] Fix mobile styles for pkey export (#7437) Raymond Jacobson
[4c205f1] Bump version to 0.6.29 audius-infra
[4140f4c] [PAY-2448] Exclude system USDC transactions by default (#7429) Randy Schott
[e68c3b7] [PROTO-1644] Healthz displays audius-d managed status and icon (#7436) Danny
[601f377] DDEX local dev  (#7435) Michelle Brier
[5149b1c] [PAY-2407][PAY-2441] Route withdraw transactions through user bank (#7388) Randy Schott
[d12cfb4] Convert history page sagas to ts (#7431) Kyle Shanks
[88abb2f] Convert search page sagas to ts (#7434) Kyle Shanks
[e00cd7e] Convert trending page sagas to ts (#7433) Kyle Shanks
[ba35a42] [PAY-2373][PAY-2406][PAY-2374] Support file sizes for stems / downloads (#7421) Raymond Jacobson
[1f5aef3] [PAY-2413] Export private key (#7425) Saliou Diallo
[57d8d7a] [PROTO-1636] Make ddex UI upload zip files to S3 (#7430) Theo Ilie
[8b5eda3] ONC-1: add metrics backfill (#7422) alecsavvy
[eb9655c] Change index_user_bank log to info instead of error (#7427) Reed
[1e09b03] [PAY-2445] Separate email notifications for manual transfer and withdrawal (#7424) Randy Schott
[5d442bd] Add orig_filename to track/stems endpoint (#7426) Reed
[72d8a04] Bump version to 0.6.28 audius-infra
[f3ebb3b] track page sagas to typescript (#7418) Andrew Mendelsohn
[5d2b7cb] Fix duplicate unregistered SPs on healthz overview (#7406) Theo Ilie
[967eeeb] Convert signon page sagas to ts (#7415) Kyle Shanks
[7bfe439] User sagas to typescript (#7405) Andrew Mendelsohn
[b70866b] Fix native "MultiselectRemove" icons (#7413) JD Francis
[0ebff33] Deprecate all non-harmony Avatar in native (#7404) JD Francis
[0e6064a] Fix profile lineup loading (#7409) Andrew Mendelsohn
[81e7bca] Small fixes for lossless upload flow (#7412) Reed
[d9f52b5] Fix @audius/common auto-imports (#7414) Dylan Jeffers
[c6c5add] Upload includes download_conditions splits, orig_filename for stems (#7411) Reed
[3c898a8] Unbundle @audius/common (#7379) Dylan Jeffers
[7d81602] Refactor to use Harmony PasswordInput in native (#7410) JD Francis
[5f911c0] Add DDEX publisher CI workflow (#7408) Michelle Brier
[ebe57b6] Fix isrc/iswc regex (#7407) Raymond Jacobson
[513b141] [PAY-2332] Add new stems and downloads upload UI (#7333) Saliou Diallo
[f083d5c] [DVRL-2] Clean out docusaurus boilerplate content (#7239) Sam Gutentag
[37f0cac] [DVRL-1]  - Prep Upgrade docusaurus to v3.1.0 (#7271) Sam Gutentag
[3d7cd2a] fix DDEX publisher dependencies and remove from blocking release (#7402) Michelle Brier
[ebf3024] [PROTO-1650] Add uptime matrix to healthz (#7400) Theo Ilie
[d09e31e] fix ddex-publisher ci (#7401) Michelle Brier
[cd8a4ed] Bump version to 0.6.27 audius-infra
[617ba71] [PAY-2440] Update indexers for cleaner withdrawal flow (#7394) Randy Schott
[79bb969] Lineup sagas to typescript (#7399) Andrew Mendelsohn
[a2571ea] Convert common js test tiles to ts (#7398) Kyle Shanks
[8ff5b73] Convert trending page store js files to ts (#7396) Kyle Shanks
[25e848c] Add maxsockets flag as workaround for constant npm ECONNRESET failures (#7397) Danny
[eddc36f] Track sagas to typescript (#7391) Andrew Mendelsohn
[234603a] Migrate (almost) all native icon references to harmony (#7377) JD Francis
[334e9d5] Convert profile page store js files to ts (#7386) Kyle Shanks
[5cd63b2] Convert track page store js files to ts (#7390) Kyle Shanks
[873fa45] [PAY-2403] Fix purchase drawer height on small iphones (#7375) Raymond Jacobson
[f85accf] Collection sagas to TS (#7387) Andrew Mendelsohn
[047c348] Fix mobile typecheck (#7389) Reed
[90d0bbf] Block future release dates on public tracks (#7358) Isaac Solo
[83670a0] [PROTO-1638] Use static nodes in uptime reporting (#7384) Theo Ilie
[2b0dd91] [PAY-2333] Download all logic for web (#7359) Reed
[72cd42b] Allow memo v2 instructions in relay (#7383) Randy Schott
[ab0ab02] DDEX publisher module scaffolding (#7381) Michelle Brier
[6808b34] Convert saved page store js files to ts (#7378) Kyle Shanks
[b801e0b] Bump version to 0.6.26 audius-infra
[16b63b3] Fix sdk not initializing on stage ddex (#7380) Michelle Brier
[4de2102] DDEX ingester scaffolding (#7361) Michelle Brier
[d363294] Convert history page store js files to ts (#7376) Kyle Shanks
[72c0af4] common/store/cache sagas to typescript (#7373) Andrew Mendelsohn
[d69458f] Convert Feed page store js files to ts (#7374) Kyle Shanks
[2e1b7bf] Convert collection page store js files to ts (#7372) Kyle Shanks
[b006c02] Fix lottie imports (#7371) Dylan Jeffers
[35e3d13] Convert common store cache js files to ts (#7369) Kyle Shanks
[cd81ec8] Select Artists page playback fixes (#7343) JD Francis
[583e266] [Web] Batch 1 of Harmony Icons Migration (#7368) nicoback2
[223cbda] [PROTO-1641] Add unregistered nodes to healthz in yellow (#7366) Theo Ilie
[c2813c1] Fix node aggregation after changes to get_all_discovery_nodes_cached (#7365) Raymond Jacobson
[d8e97ff] [C-3735] Add forceRefresh to audius-query hook results (#7345) Andrew Mendelsohn
[14d5d1a] Fix monorepo package lock (#7364) Theo Ilie
[6f15d0d] [PAY-2390] Fix chat reaction behind date + reaction popup placement (#7362) Reed
[a676c6d] Add dist and .git directories to .dockerignore to speedup build times (#7336) Danny
[2a29085] Bump version to 0.6.25 audius-infra
[facb90c] Upgrade armada cli (#7177) Raymond Jacobson
[3a0158a] [PAY-2314] Add audio user bank indexer tests (#7350) Raymond Jacobson
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants