Wrong notification preferences URL in email #4772

Closed
opened 2026-02-05 09:14:34 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @kroomagnon on GitHub (May 13, 2024).

Describe the Bug

The notification preferences URL in the email points to a 404 page.

In the mail: bookstack.example.com/preferences/notifications
On the website: bookstack.example.com/my-account/notifications

Steps to Reproduce

  1. Watch a book.
  2. Modify book (with other user).
  3. Receive notification mail.
  4. Mail contains wrong link to the notifications settings: /preferences/notifications

Expected Behaviour

Proper link in the mail: /my-account/notifications

Screenshots or Additional Context

a3a776d4a6/app/Activity/Notifications/Messages/BaseActivityNotification.php (L46)

a3a776d4a6/routes/web.php (L245C93-L245C112)

Browser Details

No response

Exact BookStack Version

v24.05

Originally created by @kroomagnon on GitHub (May 13, 2024). ### Describe the Bug The notification preferences URL in the email points to a 404 page. In the mail: bookstack.example.com/preferences/notifications On the website: bookstack.example.com/my-account/notifications ### Steps to Reproduce 1. Watch a book. 2. Modify book (with other user). 3. Receive notification mail. 4. Mail contains wrong link to the notifications settings: /preferences/notifications ### Expected Behaviour Proper link in the mail: /my-account/notifications ### Screenshots or Additional Context https://github.com/BookStackApp/BookStack/blob/a3a776d4a6d3a4d8d79c2a27a0e89dd929bd8b14/app/Activity/Notifications/Messages/BaseActivityNotification.php#L46 https://github.com/BookStackApp/BookStack/blob/a3a776d4a6d3a4d8d79c2a27a0e89dd929bd8b14/routes/web.php#L245C93-L245C112 ### Browser Details _No response_ ### Exact BookStack Version v24.05
OVERLORD added the 🐛 Bug label 2026-02-05 09:14:34 +03:00
Author
Owner

@ssddanbrown commented on GitHub (May 14, 2024):

Thanks for reporting @kroomagnon, I've assigned this for the next patch release.

@ssddanbrown commented on GitHub (May 14, 2024): Thanks for reporting @kroomagnon, I've assigned this for the next patch release.
Author
Owner

@ssddanbrown commented on GitHub (May 16, 2024):

Resolved by @KiDxS via https://github.com/BookStackApp/BookStack/pull/5008, to be part of next patch release.

@ssddanbrown commented on GitHub (May 16, 2024): Resolved by @KiDxS via https://github.com/BookStackApp/BookStack/pull/5008, to be part of next patch release.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#4772