search

SCKorea / sc-backend

스타시티즌에 대한 한국어 패치를 배포하고 게임 뉴스를 DeepL 기반으로 번역하여 제공하는 서비스
https://sc.galaxyhub.kr/
1 stars 0 forks source link

fix: excerpt 컬럼 길이 255 -> 512 변경 (#26) #35

Closed seokjin8678 closed 11 months ago

seokjin8678 commented 11 months ago

관련 이슈

PR 세부 내용

이슈 내용 그대로 excerpt 컬럼의 길이를 varchar(255) -> varchar(512)로 변경했습니다. 지금 생각해보면 news 테이블에 excerpt 컬럼이 과연 필요한지 생각이 드네요. 이것은 추후 와이어프레임을 만들며 얘기를 해봐야 할 것 같습니다.

github-actions[bot] commented 11 months ago

Test Results

10 files  10 suites   2s :stopwatch: 28 tests 28 :heavy_check_mark: 0 :zzz: 0 :x: 33 runs  33 :heavy_check_mark: 0 :zzz: 0 :x:

Results for commit f62c90c2.

:recycle: This comment has been updated with latest results.

Laeng commented 11 months ago

석진님 말씀대로 content에도 excerpt 필드가 있고 원문을 content 에 저장을 하고 사용자 언어에 따라 내용을 불러와야 하므로 중복하여 저장할 필요가 없을 것 같습니다. 깔끔하게 날려버리시죠 :D

seokjin8678 commented 11 months ago

@Laeng 우선 놔두어 보고, 와이어프레임 얘기 나눌때 확정 지어보시죠! 어떤 정보가 화면에 표시되어야 할 지 모르는 상황이라...