Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: 리트윗 시 트윗이 중복되어 보이는 문제 #65

Open
minkyu97 opened this issue Jan 8, 2022 · 5 comments
Open

Bug: 리트윗 시 트윗이 중복되어 보이는 문제 #65

minkyu97 opened this issue Jan 8, 2022 · 5 comments
Labels
bug-bounty 중간총회 버그바운티

Comments

@minkyu97
Copy link

minkyu97 commented Jan 8, 2022

실제 트위터에서는 어떤 트윗을 리트윗하더라도 해당 트윗을 리트윗했다는 표시만 되고 피드에 트윗이 중복되어 보이지 않는데 현재 2팀 와플트위터 서비스에서는 피드에 있는 임의의 트윗을 리트윗하면 해당 리트윗 객체도 피드에 나타나서 중복으로 보이는 문제가 있는 것 같습니다!

실제 트위터
image

와플트위터
image

@JYPSloth
Copy link

JYPSloth commented Jan 8, 2022

저도 리트윗관련해서 하나 제보할 게 있는데, 원래 트위터는 각 계정에 들어가서 그 계정 트윗 목록을 보면 내가 리트윗했는지 여부와 상관없이 최신트윗순으로 보입니다. 그런데 와플트위터는 제가 리트윗을 하면 제 타임라인처럼 해당 트윗이 맨 위로 갱신되어 보입니다. 그런데 원 트윗은 또 트윗목록에 그대로 남아있어, 중복 문제도 함께 있는 것 같습니다.
image

@minkyu97 minkyu97 changed the title 리트윗 시 트윗이 중복되어 보이는 문제 Bug : 리트윗 시 트윗이 중복되어 보이는 문제 Jan 8, 2022
@jhm1313 jhm1313 added the bug-bounty 중간총회 버그바운티 label Jan 8, 2022
@jhm1313
Copy link
Contributor

jhm1313 commented Jan 8, 2022

@minkyu97 트위터를 사용하다보면 retweet 객체가 보일 때도 있고 안 보일 때도 있습니다.
실제 트위터는 항상 보이게 하면 피드에 너무 많은 내용이 담길 것을 우려하여 트위터 내부의 어떤 로직에 의해 어떨 때는 보이고, 어떨 때는 안 보이게 설정을 해놓은 것으로 보입니다.
그러나 이번 토이 프로젝트에서는 이러한 로직을 하나하나 다 구현하기는 어렵다고 판단하여 전부 보이는 것으로 정했습니다.
즉, 드리고 싶은 말은 실제 트위터에서도 홈 피드에 리트윗 객체가 항상 안 보이는 것은 아닙니다!

@jhm1313
Copy link
Contributor

jhm1313 commented Jan 8, 2022

@JYPSloth 해당 부분 말씀하신대로 타인의 tweet을 retweet 시 타인의 프로필에 해당 retweet 객체가 표시되는 문제가 있습니다.
반대로 retweet을 한 유저의 프로필에는 해당 객체가 표시되지 않네요. 이 부분은 수정하도록 하겠습니다. 감사합니다!

@HunRotation profile 구현하신 계훈님 코드를 보니까 UserInfoSerializer의 get_tweets 부분에서 obj.tweets.all()을 하는 것을 볼 수 있습니다만, HomeSerializer의 get_tweets를 보시면 Q객체를 활용하여 해당 문제 해결한 부분이 있습니다.
주석에 보시면 tweets written(or replied, quoted) by me 와 tweets retweeted by me 두 개로 나누어서 tweet을 모아야 합니다.
자유 스프린트 기간에 수정해주시면 감사하겠습니다!

@jhm1313 jhm1313 changed the title Bug : 리트윗 시 트윗이 중복되어 보이는 문제 Bug: 리트윗 시 트윗이 중복되어 보이는 문제 Jan 8, 2022
@HunRotation
Copy link
Contributor

넵 두 종류의 tweet을 나눠서 모으도록 수정하겠습니다!

@gyuuuna
Copy link

gyuuuna commented Jan 8, 2022

저도 리트윗 관련하여 첨언하고 싶습니다.
구현하신 와플트위터에서는 리트윗한 트윗이 맨 위로 갱신될 때 갱신된 트윗과 원래의 트윗 모두 상단에 'Retweeted'가 표시되는데요,
실제 트위터에서는 내가 리트윗하여 맨 위로 갱신된 트윗에는 상단에 'Retweeted'가 뜨지만
맨 위로 갱신된 것이 아닌 원래의 트윗 상단에는 'Retweeted'가 뜨지 않습니다. (원래의 트윗 그대로 유지되는 것 같습니다.)
이 점 참고 부탁드립니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-bounty 중간총회 버그바운티
Projects
None yet
Development

No branches or pull requests

5 participants