카테고리 간 토픽을 조용히 이동하려다 우발적으로 이메일/알림이 대량으로 발송됨

가장 중요한 사용자들에게 이메일 폭탄을 맞으며 카테고리를 이동하는 30개의 토픽을 옮기는 과정에서 큰 피해를 입었습니다.

마지막 업데이트 이후로 대량 선택된 토픽 그룹의 카테고리가 변경될 때 알림을 억제하는 기능의 텍스트가 변경된 것을 발견했습니다. 현재 다음과 같이 표시됩니다:

저는 이 옵션을 선택하지 않았는데, 사용자에게 미친 듯이 이메일이 발송되었습니다! 대상 카테고리는 Watching first post(첫 게시물 감시)로 설정되어 있지만, 이 변경 사항 이전에는 이러한 알림이 억제되었을 것입니다.

여기서 무슨 일이 일어나고 있는 건가요?

5개의 좋아요

죄송합니다 @nathank :frowning:

로컬에서 재현을 시도해 보았고, 이 케이스를 처리했는지 확인하기 위해 스펙 커버리지를 개선했습니다…

… 하지만 모든 것이 예상대로 작동하는 것 같습니다 :thinking:

재현 단계를 찾아주실 수 있을까요?

1개의 좋아요

@zogstrip 이 케이스에 대한 테스트 커버리지를 찾을 수 없습니다.

제 이해로는 describe "silent option" 스펙은 이동 중인 토픽이 이미 감시(watched) 상태인 시나리오만 다루고 있습니다.

여기서 설명하는 케이스는 _대상 카테고리(destination category)_가 “감시 중(watched)” 또는 “첫 번째 게시물 감시(watching first post)” 상태인 시나리오를 말합니다.

@nathank, 제 지식 범위 내에서는 우리는 그런 것을 조용히(silently) 처리한 적이 없습니다.

여기서 일어나고 있는 일은, 우리가 실제로 추가한 기능의 모달이 해당 동작도 조용히 처리되어야 한다고 들리게 만든 것 같습니다. 이 모달로 인해 기대치가 변했고, 이제 그 기대를 충족하지 못하고 있는 것 같습니다.

제 추측은 그렇습니다.

이것은 정당한 Contribute > Feature 또는 Contribute > UX 요청이라고 생각하며, 이를 수행하는 것을 전적으로 지지합니다. 하지만 이것은 기술적으로 버그나 회귀(regression)라고 생각하지 않습니다.

2개의 좋아요

이것이 대량 작업 옵션에 포함된 목적의 일부라고 생각했습니다.(Disable category edit notificationsDisable tag edit notifications의 동작을 임시로 일치시키기 위함입니다).

이 옵션이 없으면 이 기능은 상당히 제한적일 것입니다.

2개의 좋아요

좋은 지적입니다. 솔직히 저도 확신이 서지 않습니다.

다만, 원래 기능 요청에서 설명한 사용 사례가 바로 이 상황이라는 점은 분명해 보입니다:

새로운 체크박스를 본 여러 사용자들도 같은 생각을 했다는 점도 알 수 있습니다: Bulk editing topic categories should not trigger thousands of email notifications - #12 by mbauman

PR 자체에서도 그것이 의도였음을 암시하는 내용이 있습니다:

“이 작업을 조용히 수행” 체크박스가 선택된 경우, :notify_category_change sidekiq 작업이 큐에 추가되지 않아야 합니다.

하지만 이 경우에 대한 명시적인 스펙(테스트) 커버리지는 찾아보기 어렵습니다.

솔직히 말씀드리면, 지난 몇 년간 매일 코드를 작성하지 않아 Ruby 스펙을 작성하는 데 약간 서툴러졌지만, 이는 잠재적인 공백으로 보입니다.

지금은 더 이상 버그처럼 느껴집니다. 회귀(regression)인지는 모르겠지만, 이 경우에 대한 커버리지가 부족한 것 같습니다.

3개의 좋아요

아하! 문제가 뭔지 찾은 것 같습니다 :bug:

@silent 옵션이 제대로 전달되지 않아서, “이 작업을 조용히 수행” 체크박스가 체크되어 있더라도 대상 카테고리를 관찰하는 사용자들이 알림을 받는 문제가 발생했습니다.

6개의 좋아요

훌륭하네요! 이 문제를 진지하게 받아들이고 매우 빠르게 수정해 주셔서 감사합니다. 방금 테스트해 봤는데 모두 정상으로 보입니다.

UI의 텍스트가 지금은 다르거나 반대로 바뀌었다는 점만 참고해 주시면 좋겠습니다. 이제 체크박스를 클릭하면 이전에 하던 것과 정반대의 동작을 수행합니다. 하지만 그 부분은 정상적으로 작동하는 것 같으니, 아마도 모든 것이 문제없이 진행되고 있을 것입니다.

이전에는 다음과 같았어요:

지금은 다음과 같아요:

1개의 좋아요

*:telephone: 메뉴 옵션이 변경되었으므로 주의 깊게 들어 주세요…"

2개의 좋아요

이 주제는 13시간 후 자동으로 닫혔습니다. 더 이상 새로운 답변을 남길 수 없습니다.