42Seoul-LastDance / Backend

2 stars 0 forks source link

userName 업데이트시 소켓을 통해서 전달 받는 정보들 업데이트 필요 #34

Closed ebang091 closed 1 year ago

ebang091 commented 1 year ago

버그 리포트

어떤 버그인가요?

rest api 를 이용하여 유저 네임을 변경했을 경우, 다음과 같은 이슈가 발생합니다.

해결 방안

OZestina commented 1 year ago

적어주신 3가지 외에도 Block user 리스트에 있는 유저 네임도 업데이트가 필요합니다 (block 해제 API가 유저 네임 기반으로 진행됩니다)

ebang091 commented 1 year ago

friendStateList를 다시 보내주고, 친구 요청 및 차단 목록을 보여주는 rest api 요청에서는 slackId를 함께 주어서, 이후 username이 변경되더라도 slackId로 요청하도록 하여 실시간에 대응할 수 있도록 하려고 합니다.

개인적으로는 처음부터 프론트도 userId로 식별하자고 했어야 했는데, 아쉬움이 남네요. 생길 수 있는 문제들에 대해서 고민이 부족했던 것 같아서 아쉽습니다ㅠ

ebang091 commented 1 year ago

restapi 변경완료!

Screen Shot 2023-10-18 at 3 36 44 PM Screen Shot 2023-10-18 at 3 37 05 PM
OZestina commented 1 year ago

F->B API 요청 중 userName으로 요청받는 API는 slackID로 요청받을 수 있도록 변경 예정입니다.

Tolerblanc commented 1 year ago

위 커밋에서 문제되었던 Rest API 모두 수정 완료했습니다. userName 변경 시 친구 리스트에서 바로 갱신 되는 것 확인했습니다. SocketEventHandler 도입 대신, 리스트를 다시 그려야 되는 이벤트를 발생시켜서 처리했습니다