Skip to content

chore: remove Home Alby Go and extension cards#2175

Open
stackingsaunter wants to merge 1 commit intomasterfrom
chore/remove-home-alby-go-extension-cards
Open

chore: remove Home Alby Go and extension cards#2175
stackingsaunter wants to merge 1 commit intomasterfrom
chore/remove-home-alby-go-extension-cards

Conversation

@stackingsaunter
Copy link
Contributor

@stackingsaunter stackingsaunter commented Mar 25, 2026

Summary

  • remove Alby Go promo card from the Home left column
  • remove Alby Browser Extension promo card from the Home left column
  • keep all other Home cards and widgets unchanged

Merge condition

  • merge only after the Stories PR is merged and Stories include both:
    • Alby Go story
    • Alby Extension story

Test plan

  • cd frontend && yarn tsc:compile
  • confirm Home renders without the two removed cards

Made with Cursor

Summary by CodeRabbit

  • Bug Fixes
    • Removed the "Alby Go" card from the home screen.
    • Removed browser extension detection and related UI prompts.

Remove the standalone Alby Go and Alby Browser Extension promo cards from the
Home left column.

Made-with: Cursor
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b51647b3-90ec-4603-aeac-56ab6121e6b7

📥 Commits

Reviewing files that changed from the base of the PR and between c0e8df6 and 281c75c.

📒 Files selected for processing (1)
  • frontend/src/screens/Home.tsx
💤 Files with no reviewable changes (1)
  • frontend/src/screens/Home.tsx

📝 Walkthrough

Walkthrough

Removed UI components and related imports from the Home screen, specifically eliminating the "Alby Go" card and "Alby Browser Extension" conditional rendering logic along with their asset imports.

Changes

Cohort / File(s) Summary
UI Component Cleanup
frontend/src/screens/Home.tsx
Deleted the Alby Go card import and link component, removed browser extension installation detection logic, and eliminated the conditional ExternalLink card that displayed when the extension was not installed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Some cards must hop away,
Their code now cleared to make way,
The extension and store link depart,
Simpler screens, a cleaner start! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: removing the Alby Go and extension promotional cards from the Home screen, which matches the actual code changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-home-alby-go-extension-cards

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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