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

[C-1430] Stop propagation on repost button click in tracks tables #2249

Merged
merged 1 commit into from
Nov 11, 2022

Conversation

Kyle-Shanks
Copy link
Contributor

Description

Stopping propagation so that clicking the repost button will not play/pause the track

Dragons

N/A

How Has This Been Tested?

Manually tested

How will this change be monitored?

N/A

Feature Flags

N/A

@Kyle-Shanks Kyle-Shanks requested review from a team and dylanjeffers and removed request for a team November 11, 2022 18:32
Copy link
Contributor

@dylanjeffers dylanjeffers left a comment

Choose a reason for hiding this comment

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

Not sure if it matters but I usually do stopprop before actions

@Kyle-Shanks
Copy link
Contributor Author

Not sure if it matters but I usually do stopprop before actions

I thought that as soon as I committed the code. Why do we normally do that? This feels like a milk before the cereal thing

@audius-infra
Copy link
Collaborator

@Kyle-Shanks Kyle-Shanks merged commit dc11096 into main Nov 11, 2022
@Kyle-Shanks Kyle-Shanks deleted the kj-stop-propagation-on-table-repost-butotn-click branch November 11, 2022 19:19
audius-infra pushed a commit that referenced this pull request Nov 12, 2022
[ad230d1] v1.4.0 (#2242) Sebastian Klingler
[dc11096] Stop propagation on repost button click in tracks tables (#2249) Kyle Shanks
[5964998] Update push notification types, update notif navigation hook, and remove userInteraction check in onNotification method (#2247) Kyle Shanks
[decabd2] [C-1461] Implement license-type/isrc upload flow (#2248) Dylan Jeffers
[9e84248] [C-1452] Implement release-date field (#2246) Dylan Jeffers
[8a0c7a8] Fix race condition checking stream_mp3 feature flag (#2244) Andrew Mendelsohn
[36d53b0] Don't delete sourcemaps on sync to s3 (#2243) 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.

3 participants