Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pending to dashboard #2250

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

heyitsmebev
Copy link
Contributor

@heyitsmebev heyitsmebev commented Jan 7, 2025

The purpose of this ticket is to update the dashboard UI to display the number of pending statuses, similar to the job page, where delivery statuses for pending, delivered, and failed messages are shown.

Ticket: #2199

These numbers should align with those provided by Jobs and Notifications table and remain consistent across both job statuses and notification statuses.

Users should see "pending" when a message is initiated, and as messages are delivered, the chart on the dashboard should update accordingly.

When testing, you may need to pull in 2199-add-pending-message-data-to-daily-and-user_daily-stats in your API repo

UI dashboard
image

@heyitsmebev heyitsmebev linked an issue Jan 7, 2025 that may be closed by this pull request
@heyitsmebev heyitsmebev self-assigned this Jan 7, 2025
@jonathanbobel
Copy link
Contributor

jonathanbobel commented Jan 15, 2025

@heyitsmebev - Small change - make the Pending color #C7CACE (same color as the remaining color above and shows up well in the colorblindness test)

const COLORS = {
delivered: '#0076d6',
failed: '#fa9441',
pending: '#808080',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#C7CACE

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.

Add pending message data to daily and user_daily stats
3 participants