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

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

Merged
merged 2 commits into from
Dec 23, 2023
Merged

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

merged 2 commits into from
Dec 23, 2023

Conversation

seokjin8678
Copy link
Contributor

관련 이슈

PR 세부 내용

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

@seokjin8678 seokjin8678 added 🚨 우선순위上 중요한 작업 🛠 수정 수정에 관한 작업 ☢️ DB 데이터베이스에 관한 작업 labels Dec 17, 2023
@seokjin8678 seokjin8678 requested a review from Laeng December 17, 2023 05:21
@seokjin8678 seokjin8678 self-assigned this Dec 17, 2023
Copy link

github-actions bot commented Dec 17, 2023

Test Results

10 files  10 suites   2s ⏱️
28 tests 28 ✔️ 0 💤 0
33 runs  33 ✔️ 0 💤 0

Results for commit f62c90c.

♻️ This comment has been updated with latest results.

@Laeng
Copy link
Member

Laeng commented Dec 17, 2023

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

@seokjin8678
Copy link
Contributor Author

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

@Laeng Laeng merged commit 7c822ea into dev Dec 23, 2023
3 checks passed
@Laeng Laeng deleted the feat/#26 branch December 23, 2023 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
☢️ DB 데이터베이스에 관한 작업 🚨 우선순위上 중요한 작업 🛠 수정 수정에 관한 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: DB의 news, content 테이블의 excerpt 컬럼의 길이를 수정한다.
2 participants