1518 feat: migrate pre-award approval alerts to notification system #11770
Annotations
1 error
|
commitlint
You have commit messages with errors
⧗ input: feat: move pre-award alerts to notification system
Wire up PreAwardApprovalAlert component in Agreement Detail page and remove
old alert implementation from Procurement Tracker tab:
Agreement.jsx changes:
- Import and add PreAwardApprovalAlert component
- Add isPreAwardAlertVisible state
- Display pre-award notifications alongside BLI change request alerts
- Alerts appear at top of all Agreement Detail tabs
AgreementProcurementTracker.jsx changes:
- Remove all SimpleAlert components (approved, declined, in review, success)
- Remove alert state variables (showSuccessAlert, showInReviewAlert, etc.)
- Remove isJustSubmittedRef logic for temporary success alerts
- Remove useEffect hooks managing alert visibility based on approval status
- Remove unused imports (useLocation, useNavigate, useSelector, ProcurementTrackerPreAwardApprovalStatus)
Pre-award status alerts now:
- Display in NotificationCenter (bell icon)
- Display at top of Agreement Detail page (all tabs)
- Auto-dismiss when clicked
- Don't reappear on refresh (notification marked as read in database)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
✖ body's lines must not be longer than 100 characters [body-max-line-length]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
⧗ input: debug: add console logging to PreAwardApprovalAlert
Add temporary debug logging to help troubleshoot why alerts aren't displaying
on Agreement Detail page.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
✖ type must be one of [build, chore, ci, docs, feat, fix, perf, refactor, revert, style, test] [type-enum]
✖ found 1 problems, 0 warnings
ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
|