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: News 컬럼 nullable 제약 수정, API 문서 필수 여부 수정 (#14) #17

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

seokjin8678
Copy link
Contributor

@seokjin8678 seokjin8678 commented Dec 12, 2023

관련 이슈

close: #14

PR 세부 내용

News 컬럼의 title을 nullable 하지 않도록 변경했습니다.
또한, API 문서에 뉴스 생성 시 excerpt 필드가 필수로 되어 있으므로 해당 사항을 수정합니다.

@seokjin8678 seokjin8678 added 📘 문서화 문서화에 관한 작업 💤 우선순위下 중요하지 않은 작업 🛠 수정 수정에 관한 작업 labels Dec 12, 2023
@seokjin8678 seokjin8678 self-assigned this Dec 12, 2023
@seokjin8678 seokjin8678 requested a review from Laeng December 12, 2023 15:10
Copy link

Test Results

  5 files    5 suites   0s ⏱️
17 tests 17 ✔️ 0 💤 0
18 runs  18 ✔️ 0 💤 0

Results for commit 4d92848.

@seokjin8678 seokjin8678 changed the title fix: News 컬럼 nullable 제약 수정, API 문서 필수 여부 수정 fix: News 컬럼 nullable 제약 수정, API 문서 필수 여부 수정 (#14) Dec 12, 2023
@Laeng Laeng merged commit 5845af1 into dev Dec 12, 2023
3 checks passed
@seokjin8678 seokjin8678 deleted the feat/#14 branch December 12, 2023 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💤 우선순위下 중요하지 않은 작업 📘 문서화 문서화에 관한 작업 🛠 수정 수정에 관한 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: News의 컬럼의 nullable 제약을 수정하고, API 문서의 필수 여부를 수정한다.
2 participants