Skip to content

feat:standardize timestampp formatting across componetns#445

Merged
codeZe-us merged 3 commits into
SafeVault:devfrom
Faith-okereke:feat/consistent-date-formatting
Apr 27, 2026
Merged

feat:standardize timestampp formatting across componetns#445
codeZe-us merged 3 commits into
SafeVault:devfrom
Faith-okereke:feat/consistent-date-formatting

Conversation

@Faith-okereke
Copy link
Copy Markdown

@Faith-okereke Faith-okereke commented Apr 26, 2026

Pull Request

Summary

Standardize timestamp formatting across all dashboard tables using date-fns

Type of Change

  • Feature
  • Bug Fix
  • Refactoring
  • Documentation
  • Tests
  • Infrastructure / CI

Linked Issues

Closes #435

Key Changes

  • New Utility Module ([date.ts]
    [formatDate()]- Basic formatting
    [formatDateRange()]- Date ranges
    [formatDateTime()] Date with time
    Other helpers for flexible formatting

  • Updated 8 Components:

Finance: NairaTransactionHistory
Team Management: Timeoff, Time Tracking, Milestone, Expense, ContractCard
Contracts: ContractHistory
Invoices: InvoiceDetailsSection
Result:

  • All dashboard tables now display clean, human-readable dates
    -Easy to maintain - change formatting in one utility file to update everywhere
    -Type-safe with date-fns library

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • My changes generate no new linting/type-checking warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have updated the documentation accordingly.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 26, 2026

@Faith-okereke is attempting to deploy a commit to the vestroll's projects Team on Vercel.

A member of the Team first needs to authorize it.

Comment thread src/utils/date.ts
@codeZe-us
Copy link
Copy Markdown
Contributor

@Faith-okereke i left a comment on your PR and also fix conflict in your branch

@Faith-okereke
Copy link
Copy Markdown
Author

@Faith-okereke i left a comment on your PR and also fix conflict in your branch

I have resolved conflicts now

@codeZe-us codeZe-us self-requested a review April 27, 2026 10:45
@codeZe-us codeZe-us merged commit 4a8e740 into SafeVault:dev Apr 27, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants