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

[BE] feat: 150자가 넘어가는 경우 ... 말줄임표 추가 #568

Merged
merged 1 commit into from
Sep 6, 2024

Conversation

donghoony
Copy link
Contributor


🚀 어떤 기능을 구현했나요 ?

  • 말줄임표를 추가했습니다. 150자가 넘는 경우에만 (잘리는 경우에만) 추가됩니다.

🔥 어떻게 해결했나요 ?

  • 변수 추가, 반환할 때 concat

📝 어떤 부분에 집중해서 리뷰해야 할까요?

  • 최선의 방법이겠죠? 🤔

📚 참고 자료, 할 말

Copy link

github-actions bot commented Sep 5, 2024

Test Results

82 tests   82 ✅  3s ⏱️
24 suites   0 💤
24 files     0 ❌

Results for commit effa5ba.

Copy link
Contributor

@skylar1220 skylar1220 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

만약 말줄임표 표기 변경이 있으면 view의 변경이 서비스에 너무 영향을 미치게 되는 상황이 아닐까 생각도 들었는데,
(비슷한 예시로 미리보기 글자수가 변경되는 것도 view의 변경이지만 역시 서비스 쪽을 바꿔야하네요.)

그렇다고 '우리는 view 변경 영향 안받게 일단 내려줄테니까 프론트가 자르든 말줄임표 붙이든 알아서해!'라고 하는 건 프론트 입장에서 너무 불친절한 백엔드의 자세이니

결론적으로 어쩔 수 없나라는 생각이 드네요..!

오늘의 단어장

줄임표 (Ellipsis)

Copy link
Contributor

@nayonsoso nayonsoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

최선의 방법이겠죠? 🤔

여기에 대해서는 **클라이언트에서 말 줄임표를 붙인다면??**에 대해 생각해봤는데,

  • 전체 본문을 주고 클라이언트가 붙이기 -> 불필요한 정보가 오고간다.
  • 받아올 글자수를 서버에게 요청하고 응답 뒤에 붙이기 -> 서버가 사용자에게 보여줄 본문 크기를 결정하는게 아니라, 클라이언트가 결정할 수 있다. 이런 점에서 유연한 API 라고 할 수도 있겠지만, 목록의 모든 미리보기에서 동일한 길이를 잘라서 보여주는 지금의 화면에서는 불필요하다.

따라서 지금이 최선이라 생각해요!

Copy link
Contributor

@Kimprodp Kimprodp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

문자열 치환의 책임을 프론트에서 담당하는 것처럼 이 부분도 역시 같은 맥락이라고 생각합니다. 당장은 서버에서 처리한다 해도.. 후에는 더 적절한 방법으로 내려주고 클라이언트 쪽에서 처리되어야 할 것 같아요!

@donghoony donghoony merged commit b02b054 into develop Sep 6, 2024
6 checks passed
@donghoony donghoony deleted the be/feature/537-ellipsis-review branch September 6, 2024 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 리뷰 목록의 리뷰 미리보기 텍스트가 긴 경우 말줄임표를 포함하도록 한다.
4 participants