[codex] Redesign app overview actions and config#12
Draft
5oir5bif wants to merge 1 commit into
Draft
Conversation
There was a problem hiding this comment.
Pull request overview
This PR redesigns the dashboard’s Apps view so each app card consolidates runtime controls (enable/disable), actions, and configuration inline, while also simplifying the top navigation to the primary entry points.
Changes:
- Simplifies the top navigation to Apps, Lark Apps, and Event.
- Updates the Apps/Status view to render per-app Action and Config panels inline within each app card.
- Exposes the existing actions catalog and card renderer for reuse from the Status view, and adds CSS to support the new layout.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| dashboard/src/tabs/Status.tsx | Renders per-app inline Action/Config panels and reuses the existing per-app pages as inline config surfaces. |
| dashboard/src/tabs/Actions.tsx | Exports the action catalog and ActionCard for reuse in the Status view. |
| dashboard/src/styles.css | Makes the Apps grid full-width and styles the inline panels inside app cards. |
| dashboard/src/components/Layout.tsx | Removes Actions/Config from the top nav and updates tab-selection logic. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
21
to
+22
| const { pathname } = useLocation(); | ||
| const current = TABS.find((t) => pathname.startsWith(t.to))?.to ?? "/status"; | ||
| const current = TABS.find((t) => pathname.startsWith(t.to))?.to ?? pathname; |
Comment on lines
+22
to
+25
| /// Apps that have a dedicated config surface. The components are the existing | ||
| /// per-app pages, rendered inline so the API contracts stay exactly the same. | ||
| const APP_CONFIGS: Record<string, ComponentType> = { | ||
| linear: Linear, |
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
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
Redesign the dashboard Apps view so app controls are consolidated into each app card.
Changes
Validation
cargo check -p larkstackpnpm checkpnpm build