ecc6bcca2b
fix: use correct API endpoint for notification delete
...
DELETE /users/{owner}/messages with {type, ids} body,
not DELETE on individual message path.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-04 10:16:50 +08:00
97ddd456dd
fix: add type field to notification read request body
...
API requires both type and ids fields for marking notifications as read.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-04 09:44:48 +08:00
49415faf54
fix: use correct API endpoint for notification read
...
GitLink uses POST /users/{owner}/messages/read with ids array,
not PATCH on individual message. Fixed code and test accordingly.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-04 09:42:26 +08:00
zhangqing
263c20801a
fix: update notification test paths to include .json suffix
...
The HTTP client automatically appends .json to API paths.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-06-02 15:24:15 +08:00
zhangqing
dee04432d2
fix: use ctx.Owner instead of global cmdutil.Owner in notification
...
Tests set Owner on RuntimeContext, not on the global cmdutil variable.
This fixes all 4 notification test failures.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-06-02 15:21:12 +08:00
zhangqing
1e32e12145
feat: add notification module + fix issue --label bug
...
- Add notification +list, +view, +read, +delete commands
- Add notification unit tests
- Register notification module in register.go
- Fix issue +create ignoring --label flag
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-06-02 15:08:53 +08:00
15972095207
zhangqing