sam
(Sam Saffron)
4월 24, 2020, 3:09오전
107
사실 저는 여기에 사이트 설정을 추가하는 데 반대합니다. 원하신다면 테마 컴포넌트를 게시할 수는 있습니다.
4개의 좋아요
아니요, 현재로는 이런 방식으로 처리할 수 있는 방법이 없습니다. 죄송합니다!
3개의 좋아요
riking
(Kane York)
4월 24, 2020, 4:51오전
109
또한, 기존 동작을 복원하는 모든 커스터마이징은 bookmarkWithReminder 사용처를 단순히 bookmark로 변경하는 모든 정리 작업이 완료될 때까지 불안정할 것입니다.
1개의 좋아요
sam
(Sam Saffron)
4월 24, 2020, 4:54오전
110
또한 언급할 가치가 있는 점은, 기존 동작 이 거의 아무도 클릭하지 않는 버튼으로 아무도 정말로 관심 갖지 않는 작업을 수행하도록 되어 있었다는 것입니다. 이를 뒷받침하는 통계 자료도 충분히 있습니다.
사람들이 … 이유를 가지고 있기 때문에 … "기존 동작 복원"에 대해 매우 반대합니다.
북마크를 정말로 싫어한다면, 그냥 북마크를 비활성화하면 되지 않을까요?
10개의 좋아요
네, 맞아요. 정말로 북마크를 없애고 싶다면 해당 게시물에서 명령 목록에서 제거하면 됩니다. 이 기능은 원래부터 거의 쓸모없는 기능이었거든요. 유일한 장점이 크로스 브라우저, 크로스 OS 환경에서의 "북마크"였는데, 이건 너무 제한적인 용도입니다.
7개의 좋아요
pnoeric
(Eric)
4월 24, 2020, 7:06오전
112
알겠습니다. 그러면 ‘테마 컴포넌트’ 경로를 따르는 것이 이 문제를 해결하는 올바른 방법인가요? 여러분을 설득하려 하지는 않겠지만… 제 환경에서 이 작업을 수행하는 방법에 대한 안내를 구하는 것이 괜찮을지 궁금합니다.
알겠습니다.
1개의 좋아요
martin
(Martin Brennan)
4월 28, 2020, 6:47오전
121
오늘은 생산적인 하루였습니다. 아래 PR들이 병합되었으며, 마지막 하나를 제외한 나머지는 모두 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.
그리고 FIX: Do not save bookmark if close (X) on modal is clicked - Pull Request #9541 - discourse/discourse - GitHub 은 곧 병합될 예정입니다. 또한 /bookmarks-with-reminders URL을 /bookmarks로 되돌리는 PR도 올려 두었는데, 내일 병합되기를 희망합니다: FIX: Change bookmarks-with-reminders URL back to bookmarks for user activity - Pull Request #9566 - discourse/discourse - GitHub .
“토픽 사용자 북마크 열 로직이 올바르지 않음” 수정 사항은 몇 시간 내에 Meta에 배포될 예정입니다. 이번 주 후반에는 이러한 모든 수정 사항을 포함하여 또 다른 베타 릴리스를 배포할 계획입니다.
18개의 좋아요
martin
(Martin Brennan)
4월 29, 2020, 2:21오전
122
추가 업데이트 – 이 두 PR은 모두 병합되었으며, 해결되지 않은 버그는 없습니다. 이제 bookmarkWithReminder를 bookmark로 되돌리는 등 변수 이름 변경 작업을 위한 최종 PR을 작성하고 있습니다.
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개의 좋아요
토픽 목록 미리보기 플러그인에서 토픽 목록을 통해 클릭 한 번으로 대화상호 없이 북마크를 생성할 수 있도록 구현했습니다. (토픽에서 여전히 리치 북마크 기능을 사용할 수 있습니다.)
핵심 변경 사항에 발맞추기 위해 노력하는 중이라 며칠간 안정적이지 않을 수 있지만, 어제는 정상적으로 작동했습니다!
5개의 좋아요
martin
(Martin Brennan)
4월 30, 2020, 12:44오전
128
마지막 PR이 이제 병합되었습니다. 하루 정도 안에 새로운 베타 버전이 출시될 것입니다. 이제 새 북마크 관련 미해결 이슈는 없습니다 .
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개의 좋아요
gingerman
(ginger man)
4월 30, 2020, 1:39오전
129
새 릴리스와 함께 사용할 수 있게 되어 기대가 큽니다. 질문이 하나 있습니다.
북마크 리마인더 알림이 이메일을 보내나요? (사용자가 로그인하지 않은 경우)
1개의 좋아요
martin
(Martin Brennan)
4월 30, 2020, 6:43오전
130
사용자 다이제스트 이메일(북마크 리마인더가 포함됨)에서 읽지 않은 고우선순위 알림의 수를 확인할 수 있지만, 그 외에는 이메일로 리마인더를 받지 않습니다.
또한 북마크를 캘린더와 동기화할 수도 있습니다: Sync Bookmark Reminders with your favorite Calendar app .
4개의 좋아요
martin
(Martin Brennan)
5월 7, 2020, 3:39오전
147
몇 시간 안에 Meta에 배포될 또 다른 변경 사항을 병합했습니다. 이제 북마크 모달에 숨겨진 옵션이 추가되었는데, 리마인더가 전송된 후 북마크를 삭제하는 기능입니다. 로컬 스토리지에 선택한 옵션을 기억하므로, 생성하는 모든 북마크에 대해 동일한 동작이 적용됩니다.
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개의 좋아요
Dannii
(Dannii)
5월 8, 2020, 2:44오후
166
사람들이 브라우저 북마크를 사용한다는 생각은 내게 꽤 재미있어서, Discourse가 여전히 북마크 기능을 갖추고 있다는 사실에 매우 기쁩니다. 여기에서 게시물을 북마크하는 일은 자주 하지만, 브라우저에 북마크를 추가하는 일은 생각조차 해본 적이 없어요. 아마 내가 좀 특이한 사람이겠죠.
3개의 좋아요
martin
(Martin Brennan)
5월 13, 2020, 1:08오전
174
여기에서 문제를 수정하는 PR을 하나 더 추가했습니다: Domain.com/bookmarks is showing domain.com/u/user/activity/bookmarks-with-reminders . 또한 북마크 목록에 북마크된 게시글의 사용자 아바타도 함께 표시되도록 했습니다. 곧 병합될 예정입니다:
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

@Ellibereth 님, 이전 게시글에서 아바타를 보고 싶다고 말씀하신 것을 알고 있습니다. @exetico 님은 URL 변경 건으로 언급드립니다.
11개의 좋아요
이 책갈피의 용도가 무엇인지에 대한 정보(What is this bookmark for)가 더 많이 활용되면 좋겠습니다. 현재 이 정보를 볼 수 있는 방법은 프로필의 전체 책갈피 목록에서 확인하거나 책갈피 자체를 여는 것뿐입니다. 드롭다운 메뉴의 책갈피 목록에 이 정보가 표시되거나, 리마인더 알림에 포함되면 더 유용할 것 같습니다.
4개의 좋아요
seanblue:
리마인더 알림에 포함되었다면.
리마인더에는 추가 정보가 없으므로, 이것이 어떻게 작동한다고 생각하시나요? 단 이것만을 위해 완전히 새로운 기능 영역을 추가하시겠습니까?
3개의 좋아요
오히려 해당 필드의 텍스트(설정된 경우)가 알림에서 토픽 제목을 대체 해야 하는지 궁금합니다. 그저 생각해보는 것입니다.
7개의 좋아요
martin
(Martin Brennan)
5월 17, 2020, 10:58오후
178
여러분의 의견에 어느 정도 동의합니다. 이 부분이 매우 명확하지 않을 수 있지만, 리마인더 알림에 마우스를 올렸을 때 "이 북마크의 용도가 무엇인지"에 대한 정보를 확인할 수 있습니다. 이미 알림을 보낼 때 해당 정보를 가지고 있지만, 이 기능만을 위해 별도의 알림 형식을 만들고 싶지 않았습니다.
9개의 좋아요