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

notice posts crud hit #103

Merged
merged 7 commits into from
Dec 15, 2023
Merged

notice posts crud hit #103

merged 7 commits into from
Dec 15, 2023

Conversation

hobiJeong
Copy link
Collaborator

Description

조회수 증가 기능 구현 했습니다.

To Reviewer

스웨거 문서 관련 문제도 수정 했습니다.

Reference Link

Related Issue Link

#66

API

Method Path 설명 작업(추가, 수정, 삭제)
PUT /api/notice-posts 게시글 조회수 증가 추가

@hobiJeong hobiJeong added Feat 새로운 기능 추가 Modify 코드 수정(기능상의 수정이 있는 경우) labels Dec 11, 2023
@hobiJeong hobiJeong self-assigned this Dec 11, 2023
@hobiJeong hobiJeong linked an issue Dec 11, 2023 that may be closed by this pull request
6 tasks

@Module({
imports: [
TypeOrmExModule.forCustomRepository([NoticePostRepository]),
CommonPostsModule.forFeature(NoticePostRepository),
Copy link
Member

Choose a reason for hiding this comment

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

#102 에서 주입받는 방법이 변경돼서 해당 PR 반영되면 수정해서 리리퀘스트 주실 수 있나요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 알겠습니다!

Copy link
Member

@rrgks6221 rrgks6221 left a comment

Choose a reason for hiding this comment

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

dev 변경사항 반영 부탁드립니다

@hobiJeong
Copy link
Collaborator Author

hobiJeong commented Dec 14, 2023

dev 변경사항 반영 부탁드립니다

@rrgks6221 추가적으로 migration 파일도 충돌나는 부분 좀 수정했습니다!
아 key값 변경 부분 develop에 머지되면 그것도 반영해서 다시 리뷰 요청 하겠습니다!

@rrgks6221
Copy link
Member

dev 변경사항 반영 부탁드립니다

@rrgks6221 추가적으로 migration 파일도 충돌나는 부분 좀 수정했습니다! 아 key값 변경 부분 develop에 머지되면 그것도 반영해서 다시 리뷰 요청 하겠습니다!

pagination key 변경은 기존 코드에 영향가지않게 작업했기 떄문에 머지해도 될듯합니다.

@hobiJeong hobiJeong merged commit 78a5108 into develop Dec 15, 2023
@hobiJeong hobiJeong deleted the feat/#66/notice_posts_crud_hit branch December 15, 2023 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 추가 Modify 코드 수정(기능상의 수정이 있는 경우)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

notice board crud
2 participants