sam
(Sam Saffron)
24. April 2020 um 03:09
107
Ich bin tatsächlich gegen eine solche Site-Einstellung hier, aber du kannst gerne eine Theme-Komponente posten, wenn du Lust hast.
4 „Gefällt mir“
Nein, derzeit ist dies nicht möglich. Entschuldigung!
3 „Gefällt mir“
riking
(Kane York)
24. April 2020 um 04:51
109
Zusätzlich wird jede Anpassung, die das alte Verhalten wiederherstellt, instabil sein, bis alle Bereinigungsarbeiten (das Ändern der Verwendungen von bookmarkWithReminder zu einfach bookmark) abgeschlossen sind.
1 „Gefällt mir“
sam
(Sam Saffron)
24. April 2020 um 04:54
110
Ich denke, es ist auch erwähnenswert, dass das alte Verhalten darin bestand, dass wir einen Button hatten, den fast niemand anklickte, der eine Aktion ausführte, um die sich eigentlich niemand wirklich kümmerte. Und wir haben jede Menge Statistiken, die das belegen.
Ich bin sehr dagegen, das „alte Verhalten wiederherzustellen“, nur weil Leute … Gründe haben …
Wenn du Lesezeichen wirklich hasst, warum schaltest du sie dann nicht einfach ab?
10 „Gefällt mir“
Ja, genau, entferne es einfach aus der Befehlsliste im Beitrag, wenn du das Lesezeichen wirklich loswerden willst. Es war ohnehin eine ziemlich nutzlose Funktion. Das Einzige, wofür sie gut war, waren plattform- und browserübergreifende „Lesezeichen
7 „Gefällt mir“
pnoeric
(Eric)
24. April 2020 um 07:06
112
Okay, also ist der Weg über die „Theme-Komponente
1 „Gefällt mir“
martin
(Martin Brennan)
28. April 2020 um 06:47
121
Heute war ein produktiver Tag. Die folgenden PRs wurden gemergt, und alle außer dem letzten sind jetzt live auf Meta:
master ← jjaffeux:no-bookmark
merged 01:19AM - 28 Apr 20 UTC
master ← jjaffeux:stop-shortcut-propagation
merged 12:33AM - 28 Apr 20 UTC
This commit reworks slightly the `toggleBookmark` and `toggleBookmarkTopic` func… tions.
The following cases have been identified:
- Pressing [f] (toggleBookmarkTopic)
- a topic list item is selected, we attempt to toggle the related topic
- a post is selected, we bookmark the associated topic
- nothing is selected, if there's a currentTopic we bookmark it
- Pressing [b] (toggleBookmark)
- a post is selected, we bookmark it
- a topic list item is selected, we attempt to toggle the related topic
- nothing is selected, if there's a currentTopic we bookmark it
Note this, commit also reduces jquery usage, a bug where the [f] shortcut was propagated to the modal input, and attempts to fix a bug when bookmarking a topic list item on the front page and the firstPost couldn't be found, current repro before fix:
- navigate to /
- press [j] to select a topic list item
- press [f]
- `Uncaught TypeError: Cannot read property 'post_number' of undefined` error appears in console, the topic is not bookmarked
master ← issue/fix-topic-user-bookmarked-column-logic
merged 06:19AM - 28 Apr 20 UTC
Make sure the topic_user.bookmarked column is set correctly when user bookmarks/… unbookmarks any post in a topic. For example, you bookmarked a post in the topic that was not the OP, the bookmark icon in the topic list would not be shown. Same if deleting a bookmark for the last bookmarked post in a topic, the bookmark icon in the topic list would not be removed.

* Previously this was only setting it to true if bookmarking the OP/topic, which was not correct -- we want to show the icon on the topic list if any post is bookmarked.
* Also set to false if unbookmarking the last bookmarked post in the topic.
Also in this PR is a migration to correct any out of sync `topic_user.bookmarked` columns, based on the new logic.
Und FIX: Do not save bookmark if close (X) on modal is clicked by martin-brennan · Pull Request #9541 · discourse/discourse · GitHub sollte bald gemergt werden. Ich habe zudem einen PR eingereicht, um die URL /bookmarks-with-reminders wieder auf /bookmarks zurückzuführen; ich hoffe, diesen morgen zu mergen: FIX: Change bookmarks-with-reminders URL back to bookmarks for user activity by martin-brennan · Pull Request #9566 · discourse/discourse · GitHub .
Die Korrektur für „Die Logik der Spalte ‚Benutzer hat Thema markiert’ war nicht korrekt
18 „Gefällt mir“
martin
(Martin Brennan)
29. April 2020 um 02:21
122
Ein weiteres Update – beide PRs sind jetzt zusammengeführt und es gibt keine offenen Fehler mehr. Ich arbeite derzeit nur noch an einem finalen PR, um Variablen usw. wie bookmarkWithReminder wieder in bookmark umzubenennen.
master ← issue/cancel-bookmark-save-if-modal-close-clicked
merged 10:45PM - 28 Apr 20 UTC
* After this change the bookmark will still be saved if clicking out of the moda… l or pressing escape
* To achieve this I implemented an initiatedBy parameter for modal closing from d-modal. If clicking on the cross it is initiated by close, if clicking out of the modal it is by click out.
* These options can then be compared in controllers consuming `onClose`
master ← issue/change-bookmarks-with-reminders-route-back-to-bookmarks
merged 12:53AM - 29 Apr 20 UTC
* Bookmarks with reminders is a core feature now, no need to have a separate URL…
* Keep around the old /u/:username/activity/bookmarks-with-reminders route for backwards compat in Ember but just redirect to user activity bookmarks.
12 „Gefällt mir“
merefield
(Robert)
29. April 2020 um 07:56
126
Ich habe ein einzelnes Klicken ohne Dialog für das Lesezeichen-Setzen aus der Themenliste im Plugin „Topic List Previews
5 „Gefällt mir“
pnoeric
(Eric)
29. April 2020 um 11:42
127
Das ist großartig, vielen Dank!
2 „Gefällt mir“
martin
(Martin Brennan)
30. April 2020 um 00:44
128
Der letzte PR wurde jetzt zusammengeführt. Wir sollten in den nächsten Tagen eine neue Beta veröffentlichen. Es gibt jetzt keine offenen Probleme mehr mit den neuen Lesezeichen .
master ← issue/rename-bookmark-with-reminder-variables
merged 12:09AM - 30 Apr 20 UTC
* Rename all instances of bookmarkWithReminder and bookmark_with_reminder to jus… t bookmark
* Delete old bookmark code at the same time
* Add migration to remove the bookmarkWithReminder post menu item if people have it set in site settings
14 „Gefällt mir“
gingerman
(ginger man)
30. April 2020 um 01:39
129
Freut uns, dass du die neue Version nutzen möchtest. Nur eine Frage:
Senden die Lesezeichen-Erinnerungsnachrichten E-Mails (falls der Benutzer nicht eingeloggt ist)?
1 „Gefällt mir“
martin
(Martin Brennan)
30. April 2020 um 06:43
130
In Ihrer Benutzer-Zusammenfassungs-E-Mail erhalten Sie eine Anzahlung über ungelesene hochpriorisierte Benachrichtigungen (zu denen Lesezeichen-Erinnerungen beitragen), aber ansonsten erhalten Sie keine weitere Erinnerung per E-Mail.
Sie können Lesezeichen auch mit Ihrem Kalender synchronisieren: Sync Bookmark Reminders with your favorite Calendar app .
4 „Gefällt mir“
martin
(Martin Brennan)
7. Mai 2020 um 03:39
147
Ich habe gerade eine weitere Änderung zusammengeführt, die innerhalb weniger Stunden live auf Meta sein wird. Im Lesezeichen-Modal gibt es nun eine versteckte Option, die das Lesezeichen löscht, sobald die Erinnerung gesendet wurde. Wir speichern Ihre Auswahl im lokalen Speicher, sodass sich dieses Verhalten bei jedem neu erstellten Lesezeichen gleich verhält.
master ← feature/optionally-delete-bookmark-when-reminder-sent
merged 03:37AM - 07 May 20 UTC
We now show an options gear icon next to the bookmark name:

When expanded we show the "delete bookmark when reminder sent" option. The value of this checkbox is saved in local storage for the user.

If this is ticked, when a reminder is sent for the bookmark the bookmark itself is deleted. This is so people can use the reminder functionality by itself.
Also remove this section from the "Edit Bookmark" modal as it just added clutter, because the user can already see they had a reminder set:

Adds a default false boolean column `delete_when_reminder_sent` to bookmarks.
9 „Gefällt mir“
Dannii
(Dannii)
8. Mai 2020 um 14:44
166
Die Idee, dass Leute Lesezeichen im Browser nutzen, amüsiert mich, daher freue ich mich sehr, dass Discourse immer noch eine Lesezeichen-Funktion hat. Ich markiere hier häufig Beiträge, aber es käme mir nie in den Sinn, ein Lesezeichen in meinem Browser hinzuzufügen. Ich bin wohl seltsam.
3 „Gefällt mir“
martin
(Martin Brennan)
13. Mai 2020 um 01:08
174
Ich habe jetzt einen weiteren PR hinzugefügt, der das Problem hier behebt Domain.com/bookmarks is showing domain.com/u/user/activity/bookmarks-with-reminders , und außerdem das Benutzer-Avatar des gebookmarkten Beitrags zur Bookmark-Liste hinzufügt. Er wird bald gemergt:
master ← issue/change-bookmarks-url-back-to-topic-list
merged 04:03AM - 13 May 20 UTC
* Changes the /bookmarks URL to show the original "topics filtered by bookmark" … list instead of redirecting to user activity bookmarks (see https://meta.discourse.org/t/domain-com-bookmarks-is-showing-domain-com-u-user-activity-bookmarks-with-reminders/149252/12)
* Add the user avatar for the user who made the post that is bookmarked

cc @Ellibereth , ich weiß, dass du in einem früheren Beitrag erwähnt hast, dass du die Avatare sehen möchtest, und @exetico für die URL-Änderung.
11 „Gefällt mir“
Es wäre schön, wenn die Informationen im Feld „Wofür ist dieses Lesezeichen?
4 „Gefällt mir“
Erinnerungen enthalten keine zusätzlichen Informationen. Wie stellst du dir das vor? Eine völlig neue Funktionsfläche nur dafür einzuführen?
3 „Gefällt mir“
Eher frage ich mich, ob der Text in diesem Feld (falls gesetzt) den Thementitel in der Benachrichtigung ersetzen sollte. Nur ein Gedanke.
7 „Gefällt mir“
martin
(Martin Brennan)
17. Mai 2020 um 22:58
178
Ich stimme dir teilweise zu. Das ist vielleicht nicht sofort offensichtlich, aber du siehst die Information „Wofür ist dieses Lesezeichen?
9 „Gefällt mir“