fix(web): trailing slash on workspace notification list API

[misterkuka]

Summary

• WorkspaceNotificationService.list() / fetchNotifications() called /users/notifications without a trailing slash.
• Backend only registers .../users/notifications/ (Django).
• Unread count (.../unread/) worked; the inbox list 404'd (often surfaces as 500 behind Traefik on self-hosted).
• Every sibling method already used a trailing slash.

Changes

• packages/services/src/workspace/notification.service.ts — append / on list()
• apps/web/core/services/workspace-notification.service.ts — same for fetchNotifications()

Fixes

Closes #9489

Test plan

• Open workspace inbox with existing notifications
• Network: GET /api/workspaces/{slug}/users/notifications/?… → 200 with results
• Badge unread count still works
• Self-hosted + Traefik no longer shows 500 for the list call

Notes

Confirmed still present on v1.3.1, v1.4.0, and preview HEAD before this change.

Summary by CodeRabbit

• Bug Fixes
  • Improved reliability when loading workspace notifications by correcting the notification listing endpoint.

[CLAassistant]

All committers have signed the CLA.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 29004c03-9bbf-4ac0-8efc-a73fb4b79ba9

📥 Commits

Reviewing files that changed from the base of the PR and between 3985693 and e511976.

📒 Files selected for processing (2)

• apps/web/core/services/workspace-notification.service.ts
• packages/services/src/workspace/notification.service.ts

---

📝 Walkthrough

Walkthrough

Both notification listing services now request trailing-slash endpoints. Existing pagination and response handling remain unchanged.

Changes

Notification endpoint alignment

Layer / File(s)	Summary
Align notification list request paths apps/web/core/services/workspace-notification.service.ts, packages/services/src/workspace/notification.service.ts	The notification listing requests now include trailing slashes. Pagination parameters and response/error handling are unchanged.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: sriramveeraghanta

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the workspace notification API trailing-slash fix.
Description check	✅ Passed	The description explains the bug, root cause, changes, affected behavior, test plan, and linked issue.
Linked Issues check	✅ Passed	The changes implement the trailing-slash fix required by issue #9489 in both notification service implementations.
Out of Scope Changes check	✅ Passed	Both one-line changes directly address issue #9489, with no unrelated code changes identified.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}