popertots/Crussell
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
368 Commits 1 Branch 0 Tags
1a3829b4d9dfbe3efd4bfa31a248b79cc8cf7f2a
Commit Graph
8 Commits
Author SHA1 Message Date
popertotsandSisyphus fb910bf60d feat(backend): migrate notification IDs from int to string and update cursor format 
Change notification ID type from int to string (CHAR(12)) to match the schema migration.

- Update AdminNotification.ID field to string
- Change cursor format from int-based to string-based for pagination
- Use validators.IsValidID instead of strconv.Atoi for notification ID validation
- Update cursor format to use RFC3339Nano for precision

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
 2026-06-20 16:58:51 +01:00
popertotsandSisyphus 81e2005114 feat(backend): update notifications handler 
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
 2026-06-18 16:26:27 +01:00
popertotsandSisyphus a38fda612e fix: harden notification pagination and ID validation 
Ultraworked with Sisyphus (https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
 2026-05-31 10:55:19 +01:00
popertots 7fc58f58d9 feat: admin notification system with priority ordering, bell icon, and /notifications page 
Two-tier notification system: new_booking (all public bookings) + pending_booking (notes/today).
Priority-sorted queue, unread count polling, enriched responses with user_name/booking_start_time.
Fix critical bug: edit_requested cleanup was broken (wrong reason string in 3 handlers).
Add 15 new tests covering priority ordering, enrichment, and notification creation flows.
Update Admin Manual, Technical Manual, and gap backlog docs.
 2026-05-16 23:41:18 +01:00
popertots a7c8837073 Tests 2026-03-02 22:17:43 +00:00
popertots 44cac94f64 Fix test setup and middleware chain - Handler tests now passing 
- Fix TestRequireRoleMiddleware by chaining RequireAuth before RequireRole (role context requirement)
- Remove unused 'strings' import from testdb.go
- Create crussell_test database in Docker setup
- Tests now properly initialize authentication context for role-based tests

Result: handlers test suite passes (13/13 tests)
Remaining failures in admin/auth/bookings/portfolio/scheduling/services/user packages need further investigation (environment setup, database constraints, endpoint initialization)
 2026-02-21 23:50:17 +00:00
popertots bbb273c192 feat(notifications): add admin notification acknowledgement on booking state changes 
- Add AcknowledgePendingBookingNotification helper for acknowledging notifications
- ConfirmBookingHandler: acknowledge pending notification when booking confirmed
- Cancel handlers: acknowledge pending notification and only create cancelled_booking notification if booking was not in pending status
- Add user_notification_preferences table with email, sms, push enabled flags
- Update cancellation logic to check original status before creating notifications
 2026-02-17 21:38:50 +00:00
popertots 67d812a9be Fix booking count filters, add admin notifications, and improve bookings 
UI
 2026-01-06 16:45:36 +00:00