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.

Sign up for GitHub

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

Jump to bottom

[BE/#102] Feat : DeletePost 구현 #103

Merged
merged 1 commit into from
Nov 21, 2023
Merged

[BE/#102] Feat : DeletePost 구현 #103

merged 1 commit into from
Nov 21, 2023

Conversation

namewhat99
Copy link
Collaborator

이슈

체크리스트

  • 게시물 삭제 구현

고민한 내용

  • update 를 한번에 처리하려고 했는데 이전 데이터를 기억하는 방식이 아니라 한번에 덮었쓰는 방식이라 findOne 으로 진행함

@namewhat99 namewhat99 added review code-review BE BE 관련 post-manage 게시글 관리 labels Nov 21, 2023
koomin1227
koomin1227 approved these changes Nov 21, 2023
View reviewed changes
Copy link
Member

@koomin1227 koomin1227 left a comment

Choose a reason for hiding this comment

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

patch 랑 비슷하게 하셨군요

@koomin1227 koomin1227 merged commit efdf87d into BE Nov 21, 2023
1 check passed
@koomin1227 koomin1227 deleted the BE-deletePost-#102 branch November 27, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@koomin1227 koomin1227 koomin1227 approved these changes

Assignees

@namewhat99 namewhat99

@koomin1227 koomin1227

Labels
BE BE 관련 post-manage 게시글 관리 review code-review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@namewhat99 @koomin1227