feat(inbox): Outlook email support, HTML rendering, connection settings redesign - #230
Merged
Conversation
…design connection settings Make the Inbox work with an Outlook connection the same way it does Gmail, render messages as real HTML, and clean up the connection settings UI. - Add a provider-agnostic email service that routes list/thread/unread/send to Gmail or Outlook based on the user's connected provider - Add a per-user Microsoft Graph mail module (list, conversation, unread, send, reply) backed by the existing per-user Outlook token helper - Fix the inbox gate: it used .maybeSingle() with no provider filter, so a user with both email and a calendar connection saw "Connect your email" - Render email bodies as sanitized HTML via DOMPurify instead of raw text, preferring the text/html part (Gmail) and HTML body (Graph) - Capture the Outlook address from the OAuth id_token, since Graph /me returns UnknownError for personal Outlook.com accounts - Redesign Connected Email/Calendar settings into per-provider cards with "Connected"/Disconnect and "+ Connect X" links; zero-state buttons unchanged Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Why
The Inbox only ever called the Gmail client, so an Outlook-connected user got a "Connect your email" wall and no mail. The connection settings also needed to match the new card-based mockup.
Notable fixes
.maybeSingle()with no provider filter — a user with email and a calendar connection (both live inemail_connections) tripped it and saw the empty state.id_tokenbecause Graph/mereturnsUnknownErrorfor personal Outlook.com accounts.Verification
bun run lintpassesbun run checkpasses (0 errors)bun run test:runpasses (717 tests; +17 new for Outlook mappers, HTML body extraction, id_token parsing)Prefer-less HTML body all confirmedTest plan
🤖 Generated with Claude Code