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

feat-be: 질문 보조 설명 필드 추가 #1001

Open
wants to merge 2 commits into
base: be/develop
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 25, 2025

목적

질문의 보조 설명을 추가하기 위한 description 필드를 추가합니다.

작업 세부사항

  • Question 엔티티에 description 추가
    • 이에 따른 DTO, 서비스 코드 수정
  • DataLoader에 description 데이터 추가
  • 테스트 코드와 API 문서에 description 필드 및 검증 추가

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


QUESTION_DESC_01

closes #10

@github-actions github-actions bot added backend 백엔드 feature 새로운 기능 labels Jan 25, 2025
@github-actions github-actions bot added this to the 스프린트 7.0 milestone Jan 25, 2025
@cutehumanS2 cutehumanS2 marked this pull request as ready for review January 25, 2025 05:06
Copy link
Contributor Author

1737781619.231579

@cutehumanS2
Copy link
Member

아무나 코드 리뷰 부탁드려요.
즐거운 설 보내십셔 ~ ㅎ.ㅎ

Copy link
Contributor

@HyungHoKim00 HyungHoKim00 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 냥인.
DB에 필드가 추가되었으니, flyway 파일도 하나 생성해야 할 것 같네요~

@@ -36,6 +36,9 @@ public class Question implements SecureResource {

private String content;

@Column(columnDefinition = "TEXT")
Copy link
Contributor

Choose a reason for hiding this comment

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

데이터타입을 TEXT로 하신 이유가 있나요?

Copy link
Contributor

@xogns1514 xogns1514 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~! 명오가 남긴 부분 말고 수정할 부분 없어보입니다
즐거운 설날 보내세요🙇🏻‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드 feature 새로운 기능
Projects
Status: 검토중
Development

Successfully merging this pull request may close these issues.

3 participants