Skip to content

Conversation

@sehwanyi
Copy link
Collaborator

@sehwanyi sehwanyi commented Jan 7, 2026

이 PR이 어떤 기능을 추가했나요?

fixed #5

게시글 CRUD 구현했습니다
spring post를 참고해서 작성해서 거의 유사합니다
페이징의 경우 게시글 작성 시간을 기준으로 만들었습니다.
크롤러가 자동으로 article 작성하는 부분은 아직 작성하지 못했습니다 현우님과 회의해야 될 듯 싶습니다
삭제의 경우 artile을 참조하는 개체들을 모두 지웠는지 확인하는 과정이 이후 필요합니다.(아직 모델이 만들어지지 않아 코드 작성 불가)

Test 부분은 구현하지 못했습니다. 지금부터 여행갈 예정이라 Test는 토요일날 마무리하여 일요일 전까지 컨펌되는지 여쭤보고 싶은 바입니다. 안된다면 시간 쪼개서 마무리하겠습니다.

모델 설계 부분에서 변수선언을 var로 하였습니다. 현우님께서 val로 선언시 에러가 발생해서 var로 고쳐 작성하였으니 확인 부탁드립니다.


주의해야 할 점이나 유의사항이 있나요?

publishedAt이 Instant 속성인데 스웨거에서 string 문자열 값이 Instant 값으로 들어가는지에 대해 에러처리가 이루어지지 않았습니다 크롤러 부분에서 처리해주셨스면 합니다.

@tteokgook1
Copy link
Collaborator

@sehwanyi 여행 가는데 재촉하기는 조금 그렇고 일요일 낮까지 하시면 될 것 같아요. 당장 고치실 건 아니니까 리뷰는 제 PR 끝나고 하겠습니다

@hyun-1210
Copy link
Collaborator

hyun-1210 commented Jan 8, 2026

@sehwanyi 제가 제대로 이해했는지 모르겠는데 스웨거에서 요청을 보냈을 때 크롤러가 읽어온 값을 Instant 값으로 변환하지 못했을 때 에러를 띄울 수 있게 처리해달라는 말씀이신가요??
val publishedAt =
try {
val localDateTime = LocalDateTime.parse(dateStr, formatter)
localDateTime.atZone(ZoneId.of("Asia/Seoul")).toInstant()
} catch (e: Exception) {
Instant.now()
}
현재는 이 코드로 이미 에러가 나면 현재 시간 Instant 값으로 들어가도록 설정했는데 그게 아니라 그냥 에러를 스웨거에 나타내길 원하시는 걸까요?

Copy link
Collaborator

@tteokgook1 tteokgook1 left a comment

Choose a reason for hiding this comment

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

@sehwanyi main 머지받고 테스트 작성해주세요
테스트는 정상 작동하는 경우 중심으로 작성하고 실패 테스트는 중요한 것 위주로 작성해주세요

@sehwanyi
Copy link
Collaborator Author

@tteokgook1

@sehwanyi
Copy link
Collaborator Author

published_at 소수점이 달라 테스트 통과 안되는 것 빼고는 테스트 추가했습니다

Copy link
Collaborator

@tteokgook1 tteokgook1 left a comment

Choose a reason for hiding this comment

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

@sehwanyi 테스트 통과되게 고쳐주세요

@sehwanyi sehwanyi changed the title ✨게시글 CRUD 구현1 ✨게시글 CRUD 구현 Jan 11, 2026
@sehwanyi
Copy link
Collaborator Author

#11

@sehwanyi sehwanyi merged commit 06fdbcf into main Jan 11, 2026
1 check passed
@sehwanyi sehwanyi deleted the article branch January 11, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

article CRUD

5 participants