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

[feat] 게시물, 채팅 신고 #44

Merged
merged 18 commits into from
Jan 13, 2024
Merged

[feat] 게시물, 채팅 신고 #44

merged 18 commits into from
Jan 13, 2024

Conversation

OJOJIN
Copy link
Contributor

@OJOJIN OJOJIN commented Jan 13, 2024

Related Issue 🪢

Summary 🌿

  • 게시글, 채팅 신고 기능을 만들었습니다.
  • 하나의 신고 테이블에서 게시물, 채팅방 신고 둘다 처리하려고도 생각해봤는데 그냥 따로 두는게 나을 것 같아서 각각의 신고테이블을 뒀습니다.

Before i request PR review 🧤

  • 확인 부탁드려요

OJOJIN added 17 commits January 13, 2024 16:14
@OJOJIN
[feat] #39 게시물 신고 entity 추가
a311480
@OJOJIN
[feat] #39 게시물 신고 repository 추가
7e42e33
@OJOJIN
[feat] #39 게시물 신고 API 컨트롤러, 서비스 코드 추가
573b6bb
@OJOJIN
[feat] #39 게시물 신고 API 요청 dto 추가
bb28d8e
@OJOJIN
[feat] #39 게시물 중복신고 에러코드 추가
72ce1a3
@OJOJIN
[feat] #39 채팅방 신고 entity 추가
c0b7eb7
@OJOJIN
[feat] #39 채팅방 신고 repository 추가
6e87300
@OJOJIN
[feat] #39 채팅방 신고 api 컨트롤러, 서비스 코드 추가
0e20294
@OJOJIN
[feat] #39 채팅방 신고 api 요청 dto 추가
8b6a130
@OJOJIN
[feat] #39 채팅방 관련 errorcode 추가
e8a9a0e
@OJOJIN
[feat] #39 게시글 신고 api 요청 dto 검증 추가
bd9d219
@OJOJIN
[feat] #39 채팅방 중복신고 확인 ErrorCode 추가
08ca90a
@OJOJIN
[fix] #39 채팅방 중복신고 확인 ErrorCode 변경
f635bbe
@OJOJIN
[fix] #39 NOT_CHATROOM_USER 에러코드 수정
ac483e4
@OJOJIN
Merge remote-tracking branch 'origin/develop' into feature/42-member
3fbabb5
@OJOJIN
Merge remote-tracking branch 'origin/feature/#40-checkList' into feat…
1c34c98 
…ure/39-report
@OJOJIN
[feat] #39 신고 컨트롤러 요청 final 추가
54571fd
@OJOJIN OJOJIN added the FEAT 새로운 기능을 추가 label Jan 13, 2024
@OJOJIN OJOJIN requested review from RyuKwanKon and ziiyouth January 13, 2024 14:50
@OJOJIN OJOJIN self-assigned this Jan 13, 2024
RyuKwanKon
RyuKwanKon approved these changes Jan 13, 2024
View reviewed changes
Copy link
Contributor

@RyuKwanKon RyuKwanKon left a comment

Choose a reason for hiding this comment

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

굿 확인했습니다. 혹시 나중에 충돌날 수 있어서 controller에 정렬 한번 부탁드릴게요

ziiyouth
ziiyouth approved these changes Jan 13, 2024
View reviewed changes
Copy link
Member

@ziiyouth ziiyouth left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍

@OJOJIN OJOJIN merged commit a524d0c into develop Jan 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ziiyouth ziiyouth ziiyouth approved these changes

@RyuKwanKon RyuKwanKon RyuKwanKon approved these changes

Assignees

@OJOJIN OJOJIN

Labels
FEAT 새로운 기능을 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] 게시물, 채팅 신고
3 participants
@OJOJIN @RyuKwanKon @ziiyouth