Skip to content

fix(quests): show Plus-locked quests in the v2 sidebar quest list - #6968

Merged
tsahimatsliah merged 1 commit into
mainfrom
fix/sidebar-locked-plus-quests
Oct 20, 2026
Merged

tsahimatsliah merged 1 commit into
mainfrom
fix/sidebar-locked-plus-quests

Conversation

@tsahimatsliah

@tsahimatsliah tsahimatsliah commented Oct 20, 2026 •

Copy link
Copy Markdown
Member

Changes

Fixes the "can't claim quests" report in dailydotdev/daily#2160.

The quest list in the v2 sidebar's Streak panel (CompactQuestRow) ignored locked. When a non-Plus user finished a Plus-slot quest, the row showed 3/3 with a check and no Claim button, so it looked like claiming was broken. In production the reporter's three unclaimed completed quests are all plusOnly, and every regular quest they completed was claimed within seconds.

  • A completed locked quest now shows a Claim with Plus button (avocado, lock icon) linking to /plus, and the row link goes to Plus as well.
  • An in-progress locked quest shows a lock next to the step count and still links to the quest's own surface, because progress keeps counting without Plus.
  • The rewards group can now shrink and wrap, so the button fits the 288px rail hover panel when a quest has three rewards.
  • Adds unit tests for both locked states. (The Quest States story was removed on main in chore(storybook): remove shipped-feature stories, verification pages, Replay and component docs #6966, so there is no story change.)

Before / after screenshots (Storybook) are in the comment below.

Events

Type event_name value
New upgrade subscription target_id: quest sidebar

Experiment

No


🤖 Opened by Smith (AI assistant) on behalf of @tsahimatsliah.

Preview domain

https://fix-sidebar-locked-plus-quests.preview.app.daily.dev

@tsahimatsliah

Copy link
Copy Markdown
Member Author

Before / after. The two Plus-quest rows are from the Sidebar row story, for a non-Plus user. Before: a completed Plus quest shows 3/3 with a check and no way to claim. After: it shows Claim with Plus, and in-progress Plus quests show a lock.

before

after

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
daily-webapp Ready Ready Preview Oct 20, 2026 1:33pm UTC
storybook Error Error Oct 20, 2026 1:33pm UTC

Request Review

The compact quest list in the v2 sidebar's Streak panel ignored the locked
flag. A non-Plus user who completed a Plus-slot quest saw 3/3 with a check
and no Claim button, which read as a broken claim (dailydotdev/daily#2160).

Completed locked quests now show a 'Claim with Plus' button that links to
/plus (logged as upgrade subscription, target 'quest sidebar'), and
in-progress locked quests show a lock next to the step count. The rewards
group can now shrink and wrap so the button fits the 288px rail hover panel
with three rewards.
@tsahimatsliah
tsahimatsliah force-pushed the fix/sidebar-locked-plus-quests branch from 6199c18 to 71ab68a Compare October 20, 2026 13:26
@tsahimatsliah
tsahimatsliah merged commit 30b3193 into main Oct 20, 2026
12 of 13 checks passed
@tsahimatsliah
tsahimatsliah deleted the fix/sidebar-locked-plus-quests branch October 20, 2026 13:34

This branch had an error being deployed

1 failed and 1 active deployments
Preview – daily-webapp — 71ab68ab Deployed Oct 20, 2026 by vercel[bot]
Preview – storybook — 71ab68ab Deployed Oct 20, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant