Skip to content

fix(shared): make empty feed CTAs navigable - #6386

Merged
rebelchris merged 1 commit into
mainfrom
eng-1907-feedback-bug-report-clicking-find-squad-button-does-not
Jul 28, 2026
Merged

fix(shared): make empty feed CTAs navigable#6386
rebelchris merged 1 commit into
mainfrom
eng-1907-feedback-bug-report-clicking-find-squad-button-does-not

Conversation

@rebelchris

@rebelchris rebelchris commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Render the Following feed empty-state CTAs as real anchors with passHref so extension cross-origin navigation has native browser behavior.
  • Apply the same Link/Button anchor fix to the Reading History empty state and opportunity popover CTA.
  • Add focused shared RTL coverage for the empty-state CTA hrefs under an extension-style absolute webapp URL.

Key decisions

  • Keep next/link so webapp-local routes can still use client-side navigation.
  • Use the existing forward-ref Button directly for Link-wrapped empty-state CTAs instead of changing EmptyScreenButton globally.

Verification

  • git diff --check origin/main
  • Targeted shared ESLint on impacted files
  • Targeted Jest specs for Following and Reading History empty screens
  • node ./scripts/typecheck-strict-changed.js

Closes ENG-1907


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1907-feedback-bug-report-cli.preview.app.daily.dev

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview Jul 28, 2026 5:26pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored Jul 28, 2026 5:26pm

Request Review

@rebelchris
rebelchris merged commit dd76e12 into main Jul 28, 2026
12 checks passed
@rebelchris
rebelchris deleted the eng-1907-feedback-bug-report-clicking-find-squad-button-does-not branch July 28, 2026 17:37
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