feat(web): add Slack connect card to home + space selector polish#1137
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 88785a8 | Commit Preview URL | Jun 21 2026, 11:51 PM |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 88785a8 | Jun 21 2026, 11:50 PM |
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Adds the 'Add Supermemory to your Slack' card (status + install) on the home dashboard, and removes the border on the space-selector trigger.
MaheshtheDev
force-pushed
the
06-18-feat_web_add_slack_connect_card_to_home_space_selector_polish
branch
from
cb42de5 to
f5022c2
Compare
MaheshtheDev
changed the base branch from
06-18-feat_web_create_org_from_settings_launches_onboarding
to
company-brain-invite-accept
|
Claude finished @MaheshtheDev's task in 1m 52s —— View job Review Complete
Overview: Adds Slack connection card to the dashboard, Company Brain home view with connections board, two-pane Flows onboarding step (Slack/Claude Code/Codex), and private-beta feature flag gating for team workspaces. Issues found: None — this looks good to ship. The implementation is clean:
Score: 10/10 |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Overview: Adds a Slack connection card to the dashboard for company brain users, with status display and OAuth install flow.
Issues found: None — this looks good to ship.
The implementation is clean:
- Race condition handling in useEffect is correct with the
activeflag - Uses
credentials: "include"appropriately for cross-origin auth - Gracefully falls back to showing "Add to Slack" if the status API fails
- OAuth uses server-side redirect (standard secure pattern)
- No XSS concerns — React auto-escapes the team name
Score: 10/10
Adds a **Company Brain** settings tab showing org-shared vs personal connection status for GitHub & Linear, with connect buttons (org scope admin-only). Backed by the new `/brain/connections` API. - Two sections: Organization (shared) and Your connections - Reuses the integrations `PillButton` styling - Role via `getActiveMember()`; org-scope connect gated to admins
There was a problem hiding this comment.
Overview: Adds a Slack connection card to the dashboard for Company Brain users, a new Company Brain connections settings tab for GitHub/Linear, and gates team onboarding behind a private-beta feature flag.
Issues found: None — this looks good to ship.
The implementation is clean:
- Race condition handling in
SlackConnectCarduseEffect is correct with theactiveflag pattern - Uses
credentials: "include"appropriately for cross-origin auth on all fetch calls - Gracefully falls back to showing "Add to Slack" if the status API fails
- OAuth flows use server-side redirects and
window.openwithnoopener(standard secure patterns) - No XSS concerns — React auto-escapes the team name
- Feature flag gating is enforced both in UI (
teamGated) and server-side (effectiveMode) for defense in depth - PostHogProvider addition is required for the
useFeatureFlagEnabledhook to work
Score: 10/10
…gs; Slack-connected toast
There was a problem hiding this comment.
Overview: Adds Slack connection card to the dashboard, Company Brain home view with connections board, two-pane Flows onboarding step (Slack/Claude Code/Codex), and private-beta feature flag gating for team workspaces.
Issues found: None — this looks good to ship.
The implementation is clean:
- Race condition handling in all async useEffects is correct with the
activeflag pattern - Uses
credentials: "include"appropriately for cross-origin auth on all fetch calls - Gracefully falls back to showing "Add to Slack" or empty states when APIs fail
- OAuth flows use server-side redirects and
window.openwithnoopener(standard secure patterns) - No XSS concerns — React auto-escapes team names and dynamic content
- Feature flag gating (
allowTeam/effectiveMode) is enforced both in UI and server-side for defense in depth - PostHogProvider wrapper is required for
useFeatureFlagEnabledhook to work correctly
Score: 10/10
2 participants
Adds the 'Add Supermemory to your Slack' card (status + install) on the home dashboard, and removes the border on the space-selector trigger.