Skip to content

Simplify GatedConditionsPill by removing token gating modal logic#14063

Merged
dylanjeffers merged 1 commit intomainfrom
claude/fix-unlock-button-modal-MEXDp
Apr 5, 2026
Merged

Simplify GatedConditionsPill by removing token gating modal logic#14063
dylanjeffers merged 1 commit intomainfrom
claude/fix-unlock-button-modal-MEXDp

Conversation

@dylanjeffers
Copy link
Copy Markdown
Contributor

Summary

Refactored the GatedConditionsPill component to remove token gating-specific modal handling, simplifying the click handler logic and reducing unnecessary dependencies.

Key Changes

  • Removed useArtistCoin hook and related token data fetching
  • Removed useBuySellModal hook and modal opening logic for token-gated content
  • Simplified handleClick callback to always invoke onClick regardless of gating type
  • Removed tracking event for token-gated purchases (kept only for USDC purchases)
  • Reduced callback dependencies from 6 to 4 items
  • Removed unused TokenGatedConditions import

Implementation Details

The component now treats token-gated and USDC-gated content more uniformly in the click handler. Token-gated content no longer triggers a buy/sell modal directly from this component - the modal logic has been moved or removed from this responsibility. The tracking event for purchase clicks is now only fired for non-token-gated (USDC) purchases.

https://claude.ai/code/session_01QcAJghLw5jpUYRUPxBY9vd

…f BuySellModal

Previously, clicking the unlock button on token-gated track tiles would
open the BuySellModal directly. Now it delegates to the onClick handler
(which opens LockedContentModal), consistent with the members-only button
behavior.

https://claude.ai/code/session_01QcAJghLw5jpUYRUPxBY9vd
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 5, 2026

⚠️ No Changeset found

Latest commit: f48eb03

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 5, 2026

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14063.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

@dylanjeffers dylanjeffers merged commit 00cab3a into main Apr 5, 2026
14 checks passed
@dylanjeffers dylanjeffers deleted the claude/fix-unlock-button-modal-MEXDp branch April 5, 2026 00:53
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.

2 participants