Skip to content

feat: empty state tracking on artist page and auction result tracking#686

Merged
JanaeHijaz merged 4 commits into
mainfrom
xander.artist_empty_states
Mar 11, 2026
Merged

feat: empty state tracking on artist page and auction result tracking#686
JanaeHijaz merged 4 commits into
mainfrom
xander.artist_empty_states

Conversation

@xander-pero
Copy link
Copy Markdown
Contributor

@xander-pero xander-pero commented Mar 4, 2026

The type of this PR is: feat

This PR resolves DIAM-363

Description

Empty state behavior for the Artist page should use the new type = emptyState subfield for these two currently untracked pathways.
Screenshot 2026-03-04 at 4 21 24 PM

Clicking an auction Result normally on the Artist or Auction Result page should fire with the example schema.

  • expanded can remain null always; this appears to be from a legacy version of the page (the last clickedAuctionResultItem event was in 2023. I made it optional so we can ignore this subfield
    • Force code I think is the tracking for this event previously
  • context_page_owner_slug can populate on Artist pages, but remain NULL on Auction Result pages.

Empty state

PR Checklist (tick all before merging)

  • If I've added a new file to the tree I've exported it from the common index.ts
  • I've added comments with examples for any new interfaces and ensured that they're in the docs
  • No platform-specific terminology has been used outside of click and tap (platform is inferred by the DB storing events)

this event stopped firing in 2023, but appeared to be for auction results on the /artist/name/auction-results page
updates clickedArticleGroup and clickedAuctionResultItem examples to match the new possible type: "emptyState" and provides examples for auction results
@xander-pero xander-pero self-assigned this Mar 4, 2026
@artsyit artsyit added the Version: Minor A deploy for new features label Mar 4, 2026
Copy link
Copy Markdown
Contributor

@JanaeHijaz JanaeHijaz left a comment

Choose a reason for hiding this comment

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

LGTM👍🏽

@JanaeHijaz JanaeHijaz merged commit 0d23a56 into main Mar 11, 2026
14 checks passed
@JanaeHijaz JanaeHijaz deleted the xander.artist_empty_states branch March 11, 2026 17:08
@artsyit
Copy link
Copy Markdown
Collaborator

artsyit commented Mar 11, 2026

🚀 PR was released in v4.348.0 🚀

@xander-pero xander-pero restored the xander.artist_empty_states branch March 17, 2026 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Version: Minor A deploy for new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants