[PAY-3210, PAY-3346] Chat blast audience tile (web)#9778
Merged
amendelsohn merged 7 commits intomainfrom Sep 18, 2024
Merged
Conversation
|
Collaborator
|
Preview this change https://demo.audius.co/am-PAY-3210 |
amendelsohn
changed the title
Collaborator
|
Preview this change https://demo.audius.co/am-PAY-3210 |
dharit-tan
reviewed
Sep 18, 2024
| }, | ||
| getRemixers: { | ||
| fetch: async ( | ||
| { userId, trackId }: { userId: ID; trackId?: string }, |
Contributor
There was a problem hiding this comment.
nice thx for doing this.
dharit-tan
reviewed
Sep 18, 2024
| }) | ||
| const { data: purchasers } = useGetPurchasers({ | ||
| userId: currentUserId!, | ||
| contentId: parseInt(chat.audience_content_id ?? '0'), |
Contributor
There was a problem hiding this comment.
if audience_content_id doesn't exist, don't we want to pass undefined in here to get the full list of purchasers who have purchased any content by current user? 0 is still a valid content ID right?
dharit-tan
reviewed
Sep 18, 2024
| /> | ||
| <span className={styles.profilePictureCount}> | ||
| {`+${formatCount(remainingUsersCount)}`} | ||
| {`${remainingUsersCount < 100 ? '+' : ''}${formatCount( |
Contributor
There was a problem hiding this comment.
nit: pull out 100 to a constant?
Contributor
There was a problem hiding this comment.
actually, maybe we want to pull this whole thing out into a messages thing?
dharit-tan
reviewed
Sep 18, 2024
|
|
||
| const { audienceCount } = useChatBlastAudienceContent({ chat }) | ||
|
|
||
| const users = useAudienceUsers(chat, USER_LIST_LIMIT + 1) |
Contributor
Author
There was a problem hiding this comment.
ah because we want one extra picture for the background of the overflow tile in the user list.
The one that says +X still needs a user behind it, so if we display 10 real ones and an overflow, we need 11 real pictures
Contributor
Author
There was a problem hiding this comment.
I'll add a comment
dharit-tan
approved these changes
Sep 18, 2024
Collaborator
|
Preview this change https://demo.audius.co/am-PAY-3210 |
audius-infra
pushed a commit
that referenced
this pull request
Sep 21, 2024
[e28b82b] Add client support for single transaction reward claiming (#9801) Marcus Pasell [1696b04] Version Packages (#9772) github-actions[bot] [1c2edae] Reapply libs split (#9817) Sebastian Klingler [c9f1142] [C-5026] Optimistic UI tweaks (web only for now) (#9816) JD Francis [7c4b196] Don't show empty chat prompt for blasts (#9815) Reed [7ef38cd] [QA-1732] Undo test code to fix unread pill check for notifications (#9814) Kyle Shanks [ad274d8] [PAY-3214] Add CreateBlast CTA to followers and supporters user lists (web) (#9804) Andrew Mendelsohn [3f862a8] UDF for chat_allowed (#9789) Steve Perkins [fa1da71] revert max width on chats (#9806) Andrew Mendelsohn [6074a68] [PAY-3216] Update upload share buttons (#9805) Andrew Mendelsohn [b9a8c0b] Revert libs split (#9802) Sebastian Klingler [8c80712] Fix merge issue (#9800) Sebastian Klingler [0ea3a11] [QA-1731] Fix key filter label (#9798) Kyle Shanks [bc28909] Separate libs from main sdk bundle (#9791) Sebastian Klingler [4d8ad95] Add comment email and push (#9784) Dylan Jeffers [734f766] [QA-1717] Fix empty unfurl messages for tracks and playlists (#9795) Kyle Shanks [2faf2e0] Add CHAT routes to static_routes (#9796) Reed [8b28228] [C-4969] Comment tombstones (#9790) JD Francis [e613a0b] [PAY-3215] ChatBlastCTA in own followers and supporters screens (#9793) Andrew Mendelsohn [123403b] [PAY-3211] Chat blast audience display (mobile) (#9792) Andrew Mendelsohn [d93fcab] [PAY-3210, PAY-3346] Chat blast audience tile (web) (#9778) Andrew Mendelsohn [32f758d] [QA-1730] Add sort back to the search query (#9787) Kyle Shanks [2359e1a] Chat fix quicktype (#9782) Steve Perkins [9be9c3a] [PAY-3209] Chat blast mobile header (#9777) Reed [9f05224] Use Harmony Artwork for ChatBlastHeader (#9785) Reed [2d838c5] [C-5024] Add comment notification (#9781) Dylan Jeffers [a6850d4] Handle dupe comment thread indexing (#9574) Isaac Solo [c27a040] [C-4826] (web) Comment action confirmation modals & toasts (#9766) JD Francis [079cd5d] Remove direct only and add noQuote error (#9780) Raymond Jacobson [68c4084] Fix usdc-usdc purchase (#9779) Raymond Jacobson [839e1e8] Fix harmony tests (#9775) Sebastian Klingler [93c216b] [PAY-3367] Mobile chat blast audience/content selection (#9683) Reed [c6bb074] [PAY-3208] Web chat blast header (#9767) Reed [bfd9c38] Version Packages (#8915) github-actions[bot] [afc5416] [C-4970, C-4825] Add comment reports (#9680) Dylan Jeffers [f38fb31] Improve mobile profile spacing (#9758) Dylan Jeffers [74ed02a] Fix typecheck (#9768) Sebastian Klingler [38a5f29] [C-1601] Fix verified badge, Improve text-link (#9757) Dylan Jeffers [b6b0827] [C-5014] Remove search input auto-correct (#9756) Dylan Jeffers [d7c8a4c] [C-4973] Add edit comment e2e (#9679) Dylan Jeffers [5c839b9] Harmonize web challenge UI (#9762) Reed [94f614d] Fix composer input error and styling for DMs (#9765) Kyle Shanks [99fb10f] [C-4971] Add comment pin e2e (#9678) Dylan Jeffers [59c2d6e] [C-4988] Update mobile TrackScreen layout for remixes + comments (#9657) Sebastian Klingler [66b1b2c] [C-4943] Add comment sort (#9672) Dylan Jeffers [be35c44] Fixing lint from bad merge (#9764) Kyle Shanks [85be4f0] [QA-1326] Update challenge button labels to match spec (#9743) Reed [8e4f02b] [C-4871] Add timestamp highlighting for composer input (#9761) Kyle Shanks [0a67586] Add tab completion for compose UI autocomplete (#9682) Kyle Shanks [85b27c4] [C-5020] Search filter fixes for FilterButton (#9763) Sebastian Klingler [921f74d] Revert "[PAY-3388] Migrate feed to SDK (#9593)" (#9754) Randy Schott [fc257e1] Fix challenge summary table (#9755) Reed [ab45d0e] [QA-1713] Add filter input to feature flag modal (#9740) Dylan Jeffers
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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
getFollowers,getSupporters, andgetPurchasersOpen questions:
How Has This Been Tested?
component looks correct and fetches the appropriate data for different blast types