nathank
(Nathan Kershaw)
7월 29, 2022, 10:01오후
1
사용자가 감시 중인 단어(watched word)를 사용자 필드에 입력하면 이상한 현상이 발생합니다.
이 예시에서는 사용자 필드 Employer(s)를 사용하고 Te Whatu Ora를 감시 중인 단어 링크로 설정한 경우입니다.
기본 상태:
필드를 Te Whatu Ora로 변경한 후:
필드 자체도 변경되었습니다:
제안
감시 중인 단어 링크가 user_fields에 적용되지 않도록 수정하는 것이 좋습니다.
다른 감시 중인 단어에 대해서는 확실하지 않습니다(테스트를 하지 않았습니다).
4개의 좋아요
jessii
(Jessii)
7월 31, 2022, 6:43오후
2
hey @nathank – 당신의 게시물을 정말 좋아해요. 항상 발견하신 내용에 대해 자세한 정보를 잘 공유해 주시죠!
저도 이렇게 하면 비슷한 현상을 볼 수 있어요. 'cow goes boo’를 감시 단어 링크(watched-word-link)로 설정했는데, 사용자 필드에 추가하면 전체 링크가 표시되더라고요. 다만 제 경우에는 감시 단어의 첫 글자도 함께 표시됩니다. 당신도 그런 현상을 보고 계신가요? 링크를 변경하고, 링크 목록에서 삭제했다가 다시 추가해 보았는데도 여전히 링크 앞에 'c’가 표시되고 있어요:
3개의 좋아요
sam
(Sam Saffron)
7월 31, 2022, 10:49오후
3
@nbianca 님이 최근 아래 링크에 따라 구현을 개선했습니다:
main ← watched_words
merged 03:15PM - 26 Jul 22 UTC
The previous method for reused the PrettyText logic which applied the
watched w… ord logic, but had the unwanted effect of cooking the text too.
This meant that regular text values were converted to HTML.
Follow up to commit 5a4c35f62714d2d72bc0ee57a10e08116bdc476a.
최신 버전을 사용 중이신가요?
1개의 좋아요
이 두 번째 문제를 재현할 수 있습니다 (버전: af912b4b0e ):
https://meta.discourse.org/ 링크와 함께 'Meta’를 /admin/customize/watched_words/action/link에 추가
사용자 정의 필드(편집 가능) 생성
사용자의 사용자 정의 필드에 'Meta’를 입력하고 저장
페이지 새로고침
'Meta’가 'Mhttps://meta.discourse.org/'로 대체됨
4개의 좋아요
nathank
(Nathan Kershaw)
8월 2, 2022, 2:04오후
6
Sam Saffron:
최신 버전을 실행 중이신가요?
네, 최신 버전을 실행 중입니다(해당 머지 후 업데이트했습니다). 제 생각에는 새로운 버그인 것 같은데 - 해당 코드 변경 사항과 관련이 있을까요?
1개의 좋아요
loic
(Loïc Guitaut)
8월 3, 2022, 3:49오후
10
로컬에서는 재현할 수 없습니다. 이 버그는 어제 병합된 이 PR로 수정되었을 것 같습니다:
main ← s3lase:feature/support-case-sensitive-watched-words
merged 08:06AM - 02 Aug 22 UTC
Currently, all Watched Words are matched case-insensitively. This presents a pro… blem for matching words where the case must be considered e.g. Brand names.
This PR introduces support for a toggle/flag(`case_sensitive`) for Watched Words and updates the matching implementation. The `flag` defaults to `false` for new and existing Watched Words.
Summary of changes:
- [X] `WatchedWord` model/table updates to handle case-sensitivity via a `case_sensitive` column/attribute.
- [X] Updates to `Admin::WatchedwordsController` to allow the creation of case-sensitive Watched Words.
- [x] Updates to WordWatcher to handle case-sensitive matching
- [x] UI updates for toggle
<img width="1194" alt="Screenshot 2022-07-18 at 6 05 06 AM" src="https://user-images.githubusercontent.com/849886/179469861-888bbdca-d080-43eb-ad72-a032e33583c9.png">
5개의 좋아요
sam
(Sam Saffron)
8월 10, 2022, 10:50오후
14
이 이슈를 닫아도 될까요? 재현 사례가 없는데, @nathank님은 최신 버전으로 재현이 가능한가요?
2개의 좋아요
sam
(Sam Saffron)
8월 16, 2022, 9:49오후
15
@nathank 이제 수정되었다고 가정하고, 여전히 문제가 있다면 다시 열기 플래그를 올려주세요.
1개의 좋아요
nathank
(Nathan Kershaw)
8월 21, 2022, 5:54오전
18
샘, 죄송합니다. 지난 몇 주가 바빴습니다.
불행히도 문제가 여전히 남아 있습니다. 재현하려면 사용자 필드를 공개 프로필에 표시?로 설정해야 합니다. 거의 기본 상태인 사이트에서도, 플러그인으로 가득 찬 일반적인 사이트에서도 이렇게 할 수 있었습니다.
대소문자 구분 설정과 무관하게 발생하며, Replace와 Link 모두에서(그리고 아마도 다른 옵션에서도, 하지만 테스트하지는 않았습니다) 발생합니다.
1개의 좋아요
MarcP
(MarcP)
11월 28, 2022, 9:48오전
19
방금 깨달은 건데, 감시 단어(watched words) > 링크 기능이 커스텀 필드에서도 작동하더라고요.
예를 들어,
discourse = https://meta.discourse.com
커스텀 프로필 필드 X의 값이 discourse인 경우, 프로필 필드에 링크가 표시됩니다.
이것은 의도된 동작이 아닐 것 같습니다.
3개의 좋아요
Hi @MarcP
최신 discourse 브랜치 test-passed에서 테스트해 보았지만 문제를 재현할 수 없습니다. 어떤 discourse 버전을 사용 중인지 공유해 주시겠어요?
MarcP
(MarcP)
11월 28, 2022, 11:22오전
21
이 문제를 보고하기 전에 재빌드를 실행했으므로, 현재 사용 중인 버전은 2.9.0.beta13 (0cb6d8c8fc )입니다.
사용자 정의 필드에서 이 문제를 발견했습니다. 필드를 편집한 후 저장을 클릭하면, 해당 단어에 대해 지정했던 URL 값이 반환됩니다.
1개의 좋아요
MarcP
(MarcP)
11월 28, 2022, 11:40오전
23
완벽합니다.
또한 "Test Custom Field"는 "Web Site"와 동일한 헤더 스타일링을 가져야 하지 않나요? 제 인스턴스에서도 동일한 UI 버그가 확인됩니다.
1개의 좋아요
MarcP
(MarcP)
1월 30, 2023, 8:39오후
33
방금 이 문제가 아직 해결되지 않았다는 걸 알아차렸어요.
1개의 좋아요