Merged
Conversation
….js charts
- Replaced makeDataset with custom createTimeSeriesData for hourly grouping
- Added vibrant colors with smooth curved lines (tension: 0.4)
- Implemented proper Chart.js time series format { x, y }
- Fixed status_code string to number conversion
- API Error Distribution now only shows POST, PUT, DELETE (no GET)
- Added beautiful hover effects and animations
- Improved chart styling with better colors and fills
- Removed unnecessary chartLabels (not needed for time series)
- Better responsive layout with fixed height charts
- Cleaner, more maintainable code structure
…nd size 1. Increased chart height from h-64 to h-96 for better visibility 2. Fixed webhook duration conversion from seconds to milliseconds (* 1000) 3. Fixed API requests not showing by displaying individual points for sparse data (< 10 records) 4. Reduced legend circle size with pointStyleWidth: 8 5. Added pointBackgroundColor for visible points in sparse data
- Added pointBorderColor and pointBorderWidth for visible points - Increased pointRadius from 4 to 5 for better visibility - Points now show without hovering when data is sparse (< 10 records) - Hover radius increased to 7 for better interaction
1. Added boxHeight: 8 to legend labels to make perfect circles (not ovals) 2. Fixed webhook duration conversion with parseFloat for accuracy 3. Updated Y-axis ticks to show decimals for values < 10ms 4. Added tooltip formatting to show 3 decimal places for small durations 5. Made duration points visible for sparse data (< 50 records)
1. Changed pointStyleWidth to boxWidth for proper circular legend markers 2. Added debug logging to investigate duration = 0 issue 3. Improved duration parsing to handle both string and number types 4. Will help identify if duration field is null/undefined/0
The duration field was being returned by the API but wasn't defined in the Ember Data model, causing it to be undefined. Added: @attr('number') duration; This will fix the webhook response time chart showing 0.000ms.
1. Increased all chart heights from h-96 (384px) to h-[28rem] (448px) 2. Changed duration chart fill from 'true' to 'origin' for proper rendering 3. Increased backgroundColor opacity from 0.15 to 0.2 for better visibility Charts now have more vertical space and duration chart has visible fill.
The h-[28rem] was too tall, reverting to original h-96 (384px). Duration chart fill improvement is kept.
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
1 participant
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.