Skip to content

feat: resolve issues #1048, #1047, #1046, #1042 - #1174

Merged
vjuliaife merged 1 commit into
vjuliaife:mainfrom
Smartdevs17:fix/all-issues
Aug 28, 2026
Merged

feat: resolve issues #1048, #1047, #1046, #1042#1174
vjuliaife merged 1 commit into
vjuliaife:mainfrom
Smartdevs17:fix/all-issues

Conversation

@Smartdevs17

Copy link
Copy Markdown
Contributor

Summary

This PR resolves 4 issues in a single changeset:

Closes #1048 — DepositWizard cancel/exit

  • Added visible Cancel button on preview and confirm steps
  • Cancel resets wizard to amount step with default state
  • Cancel hidden during in-flight deposit (busy=true)
  • Added optional prop for parent integration

Closes #1047 — Stakeholder Notification Subscriptions for Upgrade Proposals

  • New table for opt-in subscriptions
  • New table for event retention
  • API routes: POST/DELETE/GET and GET
  • Notification service alerts subscribers on propose/approve/cancel events
  • New notification kinds: , ,

Closes #1046 — Admin/Importer Annotation Notes on Bond Timeline Events

  • New table
  • API routes: POST/GET/PATCH/DELETE
  • Updated component with click-to-view annotation popover
  • Only importer and surety_admin can view/add annotations
  • Annotation edits are timestamped and attributed to author

Closes #1042 — Configurable Business Hours and SLA Tracking

  • New table with per-day hours and holidays
  • New table for configurable SLA hours per item type
  • New table for monitoring item response times
  • API routes at , , ,
  • SLA breach checker job runs every 5 minutes
  • SLA compliance rate reportable over date ranges

Changes

New Files

  • apps/api/src/migrations/0006_stakeholder_subscriptions_annotations_sla.ts — Database migration
  • apps/api/src/routes/upgrade-subscriptions.ts — Upgrade subscription API
  • apps/api/src/routes/bond-annotations.ts — Bond annotation API
  • apps/api/src/routes/sla.ts — Business hours & SLA API
  • apps/api/src/services/upgrade-notifications.ts — Notification service
  • apps/api/src/jobs/sla-breach-checker.ts — SLA breach monitoring job

Modified Files

  • apps/web/components/DepositWizard.tsx — Added cancel functionality
  • apps/web/components/BondTimeline.tsx — Added annotation popover
  • apps/web/app/app/page.tsx — Pass importerId/userRole to BondTimeline
  • apps/web/lib/api.ts — Added annotation API methods and types
  • apps/api/src/constants/notification-kinds.ts — Added upgrade + SLA notification kinds
  • apps/api/src/index.ts — Registered new routes and SLA breach checker job

…juliaife#1042

- vjuliaife#1048: Add cancel/exit controls to DepositWizard on preview and confirm steps
  - Cancel resets wizard to amount step with default state
  - Cancel hidden during in-flight deposit (busy state)
  - Added optional onCancel prop for parent integration

- vjuliaife#1047: Add stakeholder notification subscriptions for contract upgrade proposals
  - New upgrade_subscriptions table for opt-in subscriptions
  - New upgrade_notification_history table for event retention
  - API routes for subscribing/unsubscribing and viewing history
  - Notification service that alerts subscribers on propose/approve/cancel events

- vjuliaife#1046: Add admin/importer annotation notes on bond timeline events
  - New bond_timeline_annotations table
  - API routes for CRUD operations on annotations
  - Updated BondTimeline component with click-to-view annotations
  - Only importer and surety_admin can view/add annotations
  - Annotation edits timestamped and attributed to author

- vjuliaife#1042: Add configurable business hours and SLA tracking
  - New business_hours_config table with per-day hours and holidays
  - New sla_targets table for configurable SLA hours per item type
  - New sla_tracking table for monitoring item response times
  - API routes for managing business hours, targets, and tracking
  - SLA breach checker job runs every 5 minutes
  - SLA compliance rate reportable over date ranges
@drips-wave

drips-wave Bot commented Aug 28, 2026

Copy link
Copy Markdown

@Smartdevs17 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

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
tariff-shield-web Error Error Aug 28, 2026 2:06pm

@vjuliaife
vjuliaife merged commit 2b16b71 into vjuliaife:main Aug 28, 2026
1 of 2 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

2 participants