discourse/app
Martin Brennan e58e7a49f5
FIX: Bookmark reminder was clearing incorrectly (#28506)
Followup 76c56c8284

The change introduced above made it so the expired
bookmark reminders were cleared when using the bulk
action menu for bookmarks. However this also affected
clearing reminders for bookmarks when sending notifications.

When clearing bookmark reminders after sending notifications,
we take into account the auto delete preference:

* never          - The bookmark `reminder_at` date should not be cleared,
                   and the bookmark is kept.
* clear_reminder - The bookmark `reminder_at` date is cleared and
                   the bookmark is kept

The `never` option made it so "expired" bookmark reminder show
on the user's bookmark list.

This commit fixes the change from the other commit and only
forces clearing of `reminder_at` if using the bookmark bulk
action service.
2024-08-26 09:17:39 +10:00
..
assets FIX: Video uploads sometimes hang indefinitely (#28523) 2024-08-23 15:58:54 -06:00
controllers PERF: Ensure suggested topics is only loaded on last page of topic view (#28507) 2024-08-23 16:10:50 +08:00
helpers UX: Use localized time format in embedded comments (#28014) 2024-07-22 18:42:36 +08:00
jobs DEV: Allow using an AR relation as a model in services 2024-08-20 16:32:46 +02:00
mailers UX: Use a dropdown for SSL mode for group SMTP (#27932) 2024-07-18 10:33:14 +10:00
models FIX: Bookmark reminder was clearing incorrectly (#28506) 2024-08-26 09:17:39 +10:00
serializers PERF: Ensure suggested topics is only loaded on last page of topic view (#28507) 2024-08-23 16:10:50 +08:00
services DEV: Use Service::Base for suspend and silence actions (#28459) 2024-08-22 14:38:56 +03:00
views DEV: Convert account activation pages to use Ember (#28206) 2024-08-12 18:02:00 -03:00