Skip to content

feat: Up Next banner with Open/Join and meeting links - #2561

Merged
tyler-dane merged 1 commit into
mainfrom
claude/compass-event-tooltip-7abfbf
Aug 4, 2026
Merged

feat: Up Next banner with Open/Join and meeting links#2561
tyler-dane merged 1 commit into
mainfrom
claude/compass-event-tooltip-7abfbf

Conversation

@tyler-dane

@tyler-dane tyler-dane commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a centered "Starts in N minutes" banner (mirroring Vimcal) when the next event is about to start, with Open [N] to view details, Join [V] to open the meeting link when one exists, and dismiss via click or Escape
  • Consolidates the existing N "open Up Next event" shortcut into the new banner, removing the now-redundant useUpNextEventShortcut and its two view-level call sites

Note on scope

This branch was rebased onto current main after discovering that #2555/#2557/#2558 had independently landed the meeting-link (and location/attendees) plumbing through core/sync/backend, plus a Join link on the existing sidebar UpNextCard, while this branch was in progress. The redundant schema/translation work from this PR's original scope was dropped in favor of main's already-shipped version; this PR now only adds the new bottom-center banner surface and its own tests.

Test plan

  • bun run type-check clean
  • bun run lint clean (no new warnings)
  • Core (526), sync (271), backend (256) suites pass
  • Web: 33 targeted tests pass (UpNextBanner, UpNextCard, shortcuts data, WeekView render)
  • Manually verified in the browser: banner appears within the 2-minute window, Open/N opens the event form, Join/V opens the meeting link in a new tab, dismiss (click and Escape) hides it and stays hidden

🤖 Generated with Claude Code

…g links

Surfaces a centered "Starts in N minutes" banner (mirroring Vimcal) when the
next event is about to start, with Open [N] to view details or Join [V] to
open the meeting link in a new tab, dismissible via Escape or click.

Plumbs the synced conference/meeting URL through core, sync, and backend so
it reaches the browser for the first time (previously dropped at two read
translation points).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@tyler-dane
tyler-dane force-pushed the claude/compass-event-tooltip-7abfbf branch from 42ab7be to 2dbe2da Compare August 4, 2026 00:20
@tyler-dane
tyler-dane merged commit f5e3d0c into main Aug 4, 2026
20 checks passed
@tyler-dane
tyler-dane deleted the claude/compass-event-tooltip-7abfbf branch August 4, 2026 00:24
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