Skip to content

Conversation

Louis454545
Copy link
Contributor

Summary

Adds a "Sent At" timestamp column to the webhook deliveries table, allowing users to see when webhook deliveries were sent without having to expand each row.

Changes

  • Added FormattedDateTime import for proper timestamp formatting
  • Added new "Sent At" column displaying delivery.created_at
  • Enabled sorting on the created_at field for better usability
  • Updated getCellColSpan calculation from 4 to 5 columns for expanded rows
  • Used short date and time format for optimal readability

Before/After

Before: Users had to expand each webhook delivery row to see the "Sent at" timestamp

After: Timestamp is immediately visible in the main table as a dedicated column

Testing

  • Component compiles successfully
  • ESLint passes without errors
  • UI package builds successfully
  • FormattedDateTime component integration verified

Fixes #6316

- Add FormattedDateTime import for proper date formatting
- Add new 'Sent At' column showing webhook delivery timestamp
- Enable sorting on created_at field
- Update colspan calculation for expanded rows (4→5 columns)
- Display timestamp with short date and time format for readability

Fixes polarsource#6316
Copy link

vercel bot commented Aug 20, 2025

@Louis454545 is attempting to deploy a commit to the polar-sh Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Aug 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
polar-sandbox Ready Ready Preview Comment Aug 27, 2025 11:46pm

Copy link
Contributor

@psincraian psincraian left a comment

Choose a reason for hiding this comment

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

Nice contribution! Fix the comment and I will merge it

Set enableSorting to false for consistency with other columns
@Louis454545 Louis454545 requested a review from psincraian August 29, 2025 16:00
@psincraian psincraian enabled auto-merge (squash) September 3, 2025 10:16
@psincraian psincraian merged commit c2f6b80 into polarsource:main Sep 5, 2025
4 of 7 checks passed
@Louis454545 Louis454545 deleted the feature/webhook-delivery-timestamp branch September 5, 2025 15:09
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.

Webhook: Display time in the table row
2 participants