Skip to content

fix(dashboard): remove merge/duplication artifacts in App and EventEx…#421

Open
MAN7A-afk wants to merge 1 commit into
Core-Foundry:mainfrom
MAN7A-afk:fix/dashboard-corrupted-components
Open

fix(dashboard): remove merge/duplication artifacts in App and EventEx…#421
MAN7A-afk wants to merge 1 commit into
Core-Foundry:mainfrom
MAN7A-afk:fix/dashboard-corrupted-components

Conversation

@MAN7A-afk

Copy link
Copy Markdown
Contributor

Fixes corrupted dashboard files caused by a bad merge. The following files each had duplicate/interleaved component definitions from multiple concatenated versions, causing TypeScript/JSX parse errors that cascaded across the build:

  • App.tsx — contained ~5 different App component versions merged together, with duplicate export function App() blocks and incompatible imports
  • EventExplorerCard.tsx — duplicate EventExplorerCardProps interfaces and two interleaved function signatures
  • EventExplorerTable.tsx — same pattern: duplicate interface and function definitions merged together
  • EventExplorerPage.tsx — two conflicting <EventExplorerTable> usages with incompatible props interleaved in the render logic

Each file was restored to a single clean implementation, preserving existing behavior (tab URL sync in App.tsx, table/card props in EventExplorer components). Verified via static diagnostics — no remaining syntax or type errors across all four files.

Rest of the dashboard src folder (NotificationPreferencesPage, WebhookDashboardPage, ExportHistoryPage, NotificationSearchPage, TemplatePreviewDemoPage, TemplatesPage, ActivityFeed, DeliveryHeatmap, NotificationTimelineView, ThemeToggle, main.tsx) was checked and found clean — no similar corruption.

Fixes CI issue on #419

@drips-wave

drips-wave Bot commented Jul 23, 2026

Copy link
Copy Markdown

Hey @MAN7A-afk! 👋 It looks like this PR isn't linked to any issue.

If this PR is for one of the issues assigned to you as part of a Wave, please link it to ensure your contribution is tracked properly. You can do this by adding a keyword to the PR description (e.g., Closes #123), or by clicking a button below:

Issue Title
#352 [Contracts] Add Notification Template Registry Link to this issue
#372 Emit Event for Subscription Updates Link to this issue

ℹ️ Learn more about linking PRs to issues

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