병합된 주제가 더 이상 위로 올라오지 않음

기여 버그
1

3.6.0.beta2-latest(e2eb9fe93f) 버전에 버그가 유입된 것으로 보입니다. 이는 지난주 제가 업데이트를 진행했을 당시에는 존재하지 않았던 문제입니다.

참고가 될 수 있도록 업데이트 비교(diff) 링크를 공유합니다: Comparing 31701bd722b635300511e9743edb40b1133ff866...e2eb9fe93f8a338df6981bae0b378f9c9eb57555 · discourse/discourse · GitHub

주제를 병합할 때 문제가 발생합니다.

이전에는 병합 대상 주제가 위로 올라오도록(bump) 처리되었습니다.

하지만 지금은 그런 일이 발생하지 않고, 두 주제 모두 화면에서 사라져 버립니다.

아래는 21분 전에 병합한 원래 주제입니다:

IMG_0093
IMG_00932732×2048 485 KB

/latest 페이지로 가면 폐쇄된 주제는 보이지만, 병합 대상 주제는 어디에도 보이지 않습니다. :person_shrugging:

IMG_0094
IMG_00941920×1439 125 KB

마찬가지로 Travel 카테고리로 가도 해당 주제가 위로 올라온 흔적이 보이지 않습니다. :person_shrugging:

IMG_0095
IMG_00951920×1439 121 KB

만약 병합 대상 주제로 가서 Reset Bump Date를 실행하면, 병합된 마지막 게시물의 정확한 타임스탬프와 함께 /latest 페이지에 마법처럼 나타납니다.

병합 시 시간 순서 유지(chronological order) 옵션을 체크하면 이 문제가 발생하지 않는 것 같습니다.

preserve1
preserve12018×870 118 KB

preserve2
preserve22346×928 298 KB

스태프 계정과 모더레이터 계정 모두에서 시도해 보았으며, 두 계정 유형 모두에서 항상 재현됩니다.

1개의 좋아요
2

이것과 관련이 있을 것 같습니다 :thinking:

2개의 좋아요
3

여기서 문제를 시나리오를 통해 명확히 설명하겠습니다.

  • 사용자가 "Topic XYZ"라는 이름의 새 주제를 생성합니다.
  • 스태프가 해당 새 주제를 기존 "ABC Topic"이라는 주제에 병합합니다.
    – 중복 주제를 방지하기 위한 표준적인 정리 작업
  • "Topic XYZ"는 비공개(비열람) 처리되고 병합되었기 때문에 이제 공개 뷰에서 사라졌습니다.
  • "ABC Topic"은 업데이트(상단 고정/최신순 이동)되지 않았기 때문에 /latest (또는 다른 최근 목록 어디에서도) 보이지 않습니다.
  • 병합되었기 때문에, 사용자는 새 주제/게시글이 생성된 사실조차 알지 못합니다 (또는 이는 병합된 결과로 기술적으로 기존 오래된 주제에 대한 답글이 되었음에도 불구하고, 해당 주제가 1년 전 것일 수 있고 업데이트되지 않았기 때문에)

얼마나 잘 설명하고 있는지 잘 모르겠습니다 :blush:

2개의 좋아요
4

네, 문제를 잘 설명하고 있다고 생각합니다.

@christo 여기서는 몇 가지 다른 조합을 고려해야 한다고 생각합니다:

토픽 A(단일 게시글)를 토픽 B에 병합하는 경우, 다음 사례들을 고려해야 합니다.

    1. 토픽 A의 게시글이 토픽 B의 마지막 게시글보다 최신일 때
    • a. 시간순 유지가 선택된 경우
    • b. 시간순 유지가 선택되지 않은 경우
    1. 토픽 A의 게시글이 토픽 B의 마지막 게시글보다 오래되었을 때
    • c. 시간순 유지가 선택된 경우
    • d. 시간순 유지가 선택되지 않은 경우

제 생각에는 2(c)를 제외한 모든 경우에서 토픽이 상위(Bump)되어야 합니다.

2(c)를 제외한 모든 경우에서 토픽 A의 게시글이 토픽 B의 끝에 추가되므로, 병합이 발생했을 때를 기준으로 상위(Bump) 날짜가 설정되어야 한다고 생각합니다.

다만 2(d)의 경우, 관리자님이 나중에 "상위 날짜 초기화(reset bump date)"를 실행하면 어떻게 될지 정확히 확신은 없습니다. 토픽 내에서 가장 최근에 생성된 게시글(예:倒数 두 번째 게시글)의 날짜로 상위 날짜가 업데이트되는 것이 적절할 것 같습니다.

@martin 우리 둘은 다른 시나리오에서 상위(Bump) 날짜 로직에 대해 논의해 왔습니다 – 위 내용이 당신에게도 타당하게 느껴집니까?

2개의 좋아요
6

네, 이번 주에 설치한 업데이트 전까지는 항상 그랬기 때문입니다.

다시 말하지만, 지금 겪고 있는 문제는 새 토픽/게시물이 더 이상 병합된 토픽을 위로 올리지 않아서 단순히 사라져 버린다는 점입니다. 과거에는 항상 그랬거든요.

동의합니다.

1개의 좋아요
7

네, 저도 그게 맞는 것 같아요 :+1:

2개의 좋아요
8

네, 위에서 링크한 커밋에서 제가 도입한 버그인 것 같습니다. 원래 의도는 대상 토픽의 최신 게시물보다 오래된 첫 번째 게시물이 병합될 때 bumped_at 날짜가 과거로 밀리는 문제를 수정하려는 것이었습니다. 하지만 첫 번째 게시물이 더 새로운 경우의 동작까지 깨뜨린 것 같습니다.

현재 이 문제를 수정 중이며, 해결되면 여기서 업데이트하겠습니다.

1개의 좋아요
10

이 문제를 수정하기 위한 PR입니다. 승인되고 병합되면 다시 업데이트하겠습니다.

3개의 좋아요
11

@christo 빠른 수정 감사합니다 :person_bowing:

3개의 좋아요
12

좋아요, 이제 수정 사항이 병합되었습니다. 이제 의도대로 정상 작동해야 합니다.

2개의 좋아요