3 Commits

Author	SHA1	Message	Date
Martin Brennan	16799da580	FIX: Improve bookmark modal on mobile and bookmark sync rake task (#9221) ... * Improve the bookmark mobile on modal so it doesn't go all the way to the edge and the custom datetime input is easier to use * Improve the rake task for syncing so it does not error for topics that no longer exist and batches 2000 inserts at a time, clearing the array each time	2020-03-17 15:15:22 +10:00
Martin Brennan	8123538c94	DEV: Minor review fixes and fix bookmark spec logging (#9045) ... As per: https://review.discourse.org/t/fix-never-allow-custom-emoji-to-be-marked-secure-8965/9072 https://review.discourse.org/t/feature-improving-bookmarks-part-2-topic-bookmarking-8954/9038	2020-03-02 15:40:29 +10:00
Martin Brennan	e1e74abd4f	FEATURE: Improving bookmarks part 2 -- Topic Bookmarking (#8954) ... ### UI Changes If `SiteSetting.enable_bookmarks_with_reminders` is enabled: * Clicking "Bookmark" on a topic will create a new Bookmark record instead of a post + user action * Clicking "Clear Bookmarks" on a topic will delete all the new Bookmark records on a topic * The topic bookmark buttons control the post bookmark flags correctly and vice-versa Disabled selecting the "reminder type" for bookmarks in the UI because the backend functionality is not done yet (of sending users notifications etc.) ### Other Changes * Added delete bookmark route (but no UI yet) * Added a rake task to sync the old PostAction bookmarks to the new Bookmark table, which can be run as many times as we want for a site (it will not create duplicates).	2020-02-13 16:26:02 +10:00