fix: commentOptionFlag 업로드 시 역으로 변환

[kimyoungyin]

개요

백엔드와의 데이터 처리 차이로 인해 댓글 끄기 옵션을 킬 것인가에 대한 boolean값으로 변환하여 백엔드에 전달하도록 수정하였습니다

[live-small]

댓글 기능 해제 옵션 on = true

UI가 댓글 기능 해제옵션이라면, 백엔드에서 반대로 바꾸는 게 맞을 거 같은데..이렇게 변경하는 이유가 있을까요? 아니면 댓글 기능해제 옵션이 아니라 댓글 활성화로 텍스트를 바꾸면 좋을 거 같아요. default는 댓글 활성화 on으로 해두고 !

@kimyoungyin

[kimyoungyin]

@live-small

ui와 프론트단에서는 코드 상에서 문제는 없습니다. state에 저장된 해당 boolean값을 (1)전달하고 (2)백엔드에서 이를 해석하는 과정에서 문제가 생긴 것인데, 두 가지 중 가장 간편한 방식(1)으로 진행하여서 UI와 state간의 불일치 걱정은 안하셔도 좋을 것 같습니다.

물론 백엔드에서 해당 부분을 수정하는 데 번거로울 까봐 먼저 한 거긴 합니다. 프론트단에서는 !만 붙이면 되니까요

[live-small]

물론 백엔드에서 해당 부분을 수정하는 데 번거로울 까봐 먼저 한 거긴 합니다. 프론트단에서는 !만 붙이면 되니까요

맞아요! 이해는 했는데, 이 부분을 다른 사람이 읽고 작업하게 된다면 UI랑 반대로 되어있으니까 이해하기 어려울 거 같아서 UI의 텍스트를 바꿔도 되지 않을까? 라는 생각에 말씀드렸습니다!

@kimyoungyin

[kimyoungyin]

혹시 UI랑 반대로 되어있다는 게 어떤 걸 말하는 걸까요? UI랑 state는 일치하는데 'UI'랑 백엔드에 전달하는 '데이터'가 반대로 되어있다는 뜻일까요? 이게 맞다면 굳이 일치할 필요까진 있을까 싶습니다 ㅋㅋㅋ 고민되시면 전체 회의 때 한 번 얘기해봐요!

@live-small

[live-small]

@kimyoungyin

혹시 UI랑 반대로 되어있다는 게 어떤 걸 말하는 걸까요? UI랑 state는 일치하는데 'UI'랑 백엔드에 전달하는 '데이터'가 반대로 되어있다는 뜻일까요?

: 맞습니다!

일단, 제가 잘 못이해하고 있을 수 있으니까..제가 이해한 바를 적어볼게요!

• 댓글기능 해제를 true -> isCommentBlocked true 인거고, 저 버튼이 파란색으로 활성화된다
• 현재 백엔드에서 commentFlag 가 false일 때, 댓글기능 해제인거고
• 프론트는 commentFlag가 true일 때, 댓글기능 해제
• 그럼 isCommentBlocked 변수명보단, isComment로 하는 게 더 적절하지 않을까? 라는 생각이었어용
• 이게 아니라면, 댓글기능 해제 -> 댓글기능 활성화 텍스트를 바꿔도 될 거 같구요!
• isCommentBlocked로 해뒀는데, commoneFlag를 반대로해서 백엔드 api를 호출하는 게, 이 배경을 모르는 사람이 보기엔 이해하기 어려울 거란 생각에서 말씀드렸습니다!

[live-small]

백엔드랑 다르게 생각해서 이렇게 작업하게 된거니까 이런 부분을 주석으로 남겨두는 게, 나중에 이해하기 쉬울 거 같은데 혹시 어떻게 생각하시나요? @kimyoungyin