Merged
Conversation
- Change title from 'Subscription Metrics' to 'Notification User Metrics' - Add data emoji (📊) favicon with improved metadata - Add OpenGraph and Twitter card metadata - Rotate chart labels 45 degrees for 'Notifications per DAO' chart - Set default table rows to 100 and remove row selector
- Fix import path in auth route - Add QueryResultRow constraint to db query function - Remove icon.tsx file (use static SVG favicon instead)
- Add query validation to prevent write operations (INSERT, UPDATE, DELETE, etc.) - Only allow SELECT and WITH queries - Add documentation and README explaining read-only enforcement - All API routes are GET requests (except auth which only sets cookies)
- Add number formatting with thousands separators - Improve tooltip styling with rounded corners and padding - Add visual cursor highlight on hover - Enhance text styling with colors and font weights
…alues display - Replace formatter-based tooltip with custom React component - Explicitly render tooltip content with formatted numbers - Ensure tooltip shows label and value clearly on hover
- Change tooltip content from JSX element to function - Function receives tooltip props from Recharts - Add z-index to ensure tooltip appears above chart - Tooltip now properly displays formatted values on bar hover
…yling - Use proper TooltipProps type from Recharts - Convert to Tailwind CSS classes for consistent styling - Improve tooltip layout and spacing - Increase chart height and add margins for better visibility - Add bar category gap for better spacing
- Remove outdated plan markdown files - Update dashboard client and ENS resolver - Add ENS resolver tests
feat: add dashboard metrics
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
2 participants
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.
No description provided.