Skip to content

Shared dates, UI primitives, coverage gate, and golden e2e - #807

Merged
amina69 merged 7 commits into
amina69:mainfrom
Ennyhorla1:wave/ennyhorla1-petad-676-675-667-666
Oct 29, 2026
Merged

amina69 merged 7 commits into
amina69:mainfrom
Ennyhorla1:wave/ennyhorla1-petad-676-675-667-666

Conversation

@Ennyhorla1

Copy link
Copy Markdown
Contributor

Summary

  • Add formatDate, formatDateTime, formatRelative, daysBetween, and isPast with calendar-date handling and tests.
  • Add shared StatusBadge, EmptyState, Skeleton, and ConfirmDialog, and use them in place of the local approval copies.
  • Turn on v8 coverage thresholds (75% global, 80% for src/features/**), upload the HTML report, and comment the diff on pull requests.
  • Add independent golden-path Playwright specs for adoption, custody, disputes, and documents.

Closes #676
Closes #675
Closes #667
Closes #666

Test plan

  • npm run test:coverage
  • npm run test:e2e for e2e/golden
  • Confirm CI fails when coverage drops under the thresholds

Made with Cursor

…ath e2e specs.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Oct 21, 2026

Copy link
Copy Markdown

@Cjay-Cyber-2 is attempting to deploy a commit to the sheriffamina1-9497's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Oct 21, 2026

Copy link
Copy Markdown

@Ennyhorla1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

The validate suite failed when GET /api/notifications returned a pre-existing unread item ahead of the emitted envelope.
@amina69

amina69 commented Oct 22, 2026

Copy link
Copy Markdown
Owner

kindly fix the conflict

# Conflicts:
#	src/components/adoption/ApprovalHistoryTab.tsx
# Conflicts:
#	src/components/adoption/ApprovalHistoryTab.tsx
# Conflicts:
#	src/features/notifications/__tests__/mockNotificationStream.test.ts
@amina69
amina69 merged commit 33d8222 into amina69:main Oct 29, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants