feat: Unify connections into the Add Memory modal; redesign disconnect dialog#900
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 9584d98 | May 05 2026, 10:34 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 9584d98 | Commit Preview URL | May 05 2026, 10:35 PM |
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. |
|
Claude finished @MaheshtheDev's task in 1m 50s —— View job Review Complete
Overview: Unifies the connections UI into the Add Memory modal, deletes the redundant Issues found: None — this looks good to ship. 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: Unifies the connections UI into the Add Memory modal, deletes the redundant ConnectionsDetail view, and simplifies the disconnect dialog from radio buttons to a single checkbox.
Issues found: None — this looks good to ship.
The refactoring is clean:
- The
ConnectionRowcomponent was properly migrated from the deleted file - The new dropdown-based "Add a connection" CTA in the connected state is a good UX improvement
- The simplified disconnect dialog correctly handles the
hasMemoriesedge case - URL routing changes from
?view=connectionsto?add=connectare consistent - The
formatRelativeTimehelper handles null/undefined inputs properly
Score: 10/10
Merge activity
|
…t dialog (#900) - Consolidate Drive/Notion/OneDrive management into the Add Memory → Connect tab and remove the standalone `view=connections` page (deleted `connections-detail.tsx`; redirected the Integrations tile and onboarding spotlight to `?add=connect`). - Replace the duplicated provider tile row when connections exist with a single `+ Add a connection` dropdown styled to match the home-page space selector. - Rebuild the disconnect dialog as a single-decision flow: keep-memories is the default, with an inline `(optional)` checkbox to also delete imported memories — primary button label/color flips to red when opted in. - Memory-of-day card now deep-links the source document via `?doc=<id>` instead of just routing to the memories list. - Refactored the Pro paywall on the connections surface into a focused, contained card matching the rest of the dashboard's visual language.
graphite-app
Bot
force-pushed
the
05-05-feat_unify_connections_into_the_add_memory_modal_redesign_disconnect_dialog
branch
from
6a286f8 to
9584d98
Compare
2 participants
Consolidate Drive/Notion/OneDrive management into the Add Memory → Connect tab and remove the standalone
view=connectionspage(deleted
connections-detail.tsx; redirected the Integrations tile and onboarding spotlight to?add=connect).Replace the duplicated provider tile row when connections exist with a single
+ Add a connectiondropdown styled to match thehome-page space selector.
Rebuild the disconnect dialog as a single-decision flow: keep-memories is the default, with an inline
(optional)checkbox toalso delete imported memories — primary button label/color flips to red when opted in.
Memory-of-day card now deep-links the source document via
?doc=<id>instead of just routing to the memories list.Refactored the Pro paywall on the connections surface into a focused, contained card matching the rest of the dashboard's visual
language.