griffin
(Jonathan Griffin)
7월 1, 2021, 3:43오후
1
두 개의 배지가 이미 선택되어 있습니다.
배지를 선택 해제하려고 한 번 클릭했습니다.
작동하지 않았습니다.
같은 배지를 선택 해제하려고 다시 클릭했습니다.
이번에는 작동했습니다.
새로운 배지(이번 달의 신인 사용자)를 선택하려고 한 번 클릭했습니다.
선택을 시도하며 여러 번 클릭해도 동일한 결과가 나옵니다. 또한 FAILED 모달이 표시될 때 Discourse 배너가 그 아래에서 사라지는 것도 확인했습니다.
4개의 좋아요
griffin
(Jonathan Griffin)
7월 1, 2021, 4:01오후
2
죄송합니다, @riking . 어떻게 된 일인지 모르겠지만, 제가 우연히 제목 수정을 건드려 버렸습니다. 어떻게 그렇게 된 건지 전혀 모르겠고, 의도적으로 한 것도 아닙니다. 혹시 또 다른 버그일까요?
그 후에 저는 수동으로 제목을 바로잡았습니다. 어떻게 된 일인지 모르겠지만, 의도하지도 않았는데 원래 제목으로 되돌려 놓은 것 같습니다.
1개의 좋아요
griffin
(Jonathan Griffin)
7월 5, 2021, 1:15오전
3
버그가 계속되고 있습니다. 이제 즐겨찾기 배지를 선택하거나 선택 해제할 수 없게 되어, 현재 표시되는 배지가 그대로 고정되어 있습니다.
2개의 좋아요
griffin
(Jonathan Griffin)
7월 6, 2021, 3:13오전
5
그들은 움직이지 않아, 움직이지 않아
더 이상 배지를 선택할 수 없어
그들은 움직이지 않아, 움직이지 않아
둘 다 바닥에 얼어붙어 버렸어
그들은 영원히 여기 남아 있을 거야
클릭이 계속 난무하게 내버려 두자
언젠가 해결책이 나올 거라고 희망해
2개의 좋아요
th21
7월 10, 2021, 11:31오후
9
1회 클릭 시 배지가 선택 상태로 변경되지만, 별 아이콘은 “채워진” 상태로 표시되지 않습니다. 페이지를 새로고침하면 배지가 선택되어 있었고, 사용자 카드에 올바르게 표시되었음을 확인할 수 있습니다.
하지만 이 표시 문제 때문에 사용자가 2회 클릭하면 별 아이콘이 채워진 상태로 표시되며, 이는 실제로 선택을 취소하는 동작입니다.
메타에서도 동일한 문제가 발생하는 것을 확인했습니다.
2개의 좋아요
피드백 감사합니다. 해당 업무를 엔지니어에게 배정했으며, 완료 시 다시 보고드리겠습니다.
3개의 좋아요
griffin
(Jonathan Griffin)
7월 12, 2021, 11:52오후
11
좀 심하시네요…
그래도 곧 나올 수정에 관여해 주신 모든 분들께 미리 감사드립니다.
2개의 좋아요
andrei
(Andrei Prigorshnev)
7월 14, 2021, 3:18오후
12
이 PR은 즐겨찾기 배지 선택 시 발생하는 문제를 수정합니다:
master ← fix/favorite-badges-problem
merged 10:51AM - 15 Jul 21 UTC
Choosing favorite badges started [behaving strange](https://meta.discourse.org/t… /failed-error-selecting-favorite-badges/195657) recently.
This PR fixes the problem.
I'm a bit concerned about API method, and I'd like to change it in subsequent PR if there are no objections. We have this method:
```
PUT user_badges/20/toggle_favorite
```
And here is JSON this method returns:
<img width="400" alt="Screenshot 2021-07-14 at 16 54 19" src="https://user-images.githubusercontent.com/1274517/125640390-0be207d6-9844-4dd9-ac75-6e12d262d7d8.png">
It's quite a lot. In fact, this method could be returning only status code. That would be enough to figure out everything on the client side. No need to serialize objects, no need to send them to the client. It'll be a bit more efficient, but I'm also concerned about clearness of API. Looks like there's no need to return anything from this method.
Even more, instead of `toggle_favorite` it can be an idempotent method that sets `is_favorite` to provided value, and returns only status code as well
```
PUT user_badges/20/favorite
{ true }
or
PUT user_badges/20/favorite
{ false }
```
1개의 좋아요
griffin
(Jonathan Griffin)
7월 14, 2021, 4:44오후
14
동작하는 것을 확인했습니다!
다만, 별을 한 번만 클릭해야 하고, 선택된 수가 업데이트되지 않는다는 사실을 무시한 채 페이지를 새로고침해야 올바른 결과를 볼 수 있다는 점을 기억해야 합니다. 별을 두 번 클릭해 선택이 해제된 것처럼 보이게 하면 상단의 수가 감소하지만, 새로고침하면 수가 원래대로 돌아가고 배지는 여전히 선택된 상태입니다.
결국, 별을 한 번 클릭하는 것만으로는 페이지 상태가 새로고침되지 않는 반면, 두 번 클릭하면 새로고침됩니다. 그러나 올바른 내부 기능은 한 번 클릭하고 수동으로 새로고침했을 때만 제공됩니다.
1개의 좋아요
andrei
(Andrei Prigorshnev)
7월 14, 2021, 4:52오후
15
griffin:
별을 한 번만 클릭해야 하고, 선택된 수가 업데이트되지 않는다는 사실은 무시하고, 올바른 결과를 보려면 페이지를 새로고침해야 한다는 점을 기억해야 합니다. 별을 두 번 클릭해 선택 해제된 것처럼 보이게 하면 상단의 수가 감소하지만, 새로고침하면 수가 원래대로 돌아오고 배지는 여전히 선택된 상태입니다.
아, 문제를 재현할 수 있습니다. 이제 보니 클라이언트 측의 문제네요. 제이슨, 보고해 주셔서 감사합니다! 이 문제를 처리하고 이 주제에서 업데이트를 게시하겠습니다.
1개의 좋아요
andrei
(Andrei Prigorshnev)
7월 14, 2021, 5:30오후
17
음, 잠깐, 아직 병합하지 않았어요. 내일 병합할 거예요. @griffin 제 PR이 당신이 설명하는 동작을 수정할 거예요. 어쨌든 한 번 더 확인해볼게요.
1개의 좋아요
griffin
(Jonathan Griffin)
7월 14, 2021, 5:54오후
18
andrei:
음, 잠깐, 아직 병합하지 않았네.
흥미롭네요…
그러면 기능이 실제로는 있었지만, 겉모습만 그렇게 보였을 수도 있겠네요. 어쨌든, 여러분의 PR이 이 버그를 영원히 해결해 주길 바랍니다!
1개의 좋아요
andrei
(Andrei Prigorshnev)
7월 15, 2021, 10:56오전
19
네, 이제 병합되었습니다. 모든 것을 업데이트한 후에는 정상적으로 작동할 것입니다.
1개의 좋아요
andrei
(Andrei Prigorshnev)
에 닫힘
7월 26, 2021, 8:00오전
24
이 주제는 10일 후 자동으로 닫혔습니다. 더 이상 답변이 허용되지 않습니다.