-
Notifications
You must be signed in to change notification settings - Fork 43
Allow track owner to share premium content to socials C-2424 #3177
base: main
Are you sure you want to change the base?
Conversation
@@ -291,12 +300,40 @@ export const useShareToStory = ({ | |||
dispatch(setCancel(() => cancelStory(platform))) | |||
toggleProgressDrawer(true, platform) | |||
|
|||
const encodedTrackId = encodeHashId(content.track.track_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
basically copied the logic from the Audio
component for getting the right query params for the stream track URL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Would it make sense to break this out into a helper function that both places use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm yeah i think i can break out some of it
@@ -291,12 +300,40 @@ export const useShareToStory = ({ | |||
dispatch(setCancel(() => cancelStory(platform))) | |||
toggleProgressDrawer(true, platform) | |||
|
|||
const encodedTrackId = encodeHashId(content.track.track_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Would it make sense to break this out into a helper function that both places use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me!
Description
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.