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] 방 CLOSE 상태 이후 피드백 개수 증가 로직 수정(#711) #719

Merged
merged 1 commit into from
Nov 5, 2024
Merged

[BE] 방 CLOSE 상태 이후 피드백 개수 증가 로직 수정(#711) #719

merged 1 commit into from
Nov 5, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 5, 2024 •
edited by jcoding-play
Loading

📌 관련 이슈

✨ PR 세부 내용

기존엔 방이 종료 되면 피드백을 작성하지 못했었습니다.

근데 정책이 바뀌고 피드백을 작성할 수 있게 되면서 방 CLOSE 상태 이후 피드백을 작성해도 피드백 받은 개수가 증가하지 않는 문제를 발견할 수 있었습니다.

이 문제를 해결했습니다~

@github-actions github-actions bot added BE 백엔드 개발 관련 작업 기능 기능 구현 작업 labels Nov 5, 2024
@github-actions GitHub Actions
Copy link
Contributor Author

github-actions bot commented Nov 5, 2024

Test Results

 59 files   59 suites   7s ⏱️
191 tests 175 ✅ 16 💤 0 ❌
198 runs  182 ✅ 16 💤 0 ❌

Results for commit 6f43118.

hjk0761
hjk0761 approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@hjk0761 hjk0761 left a comment

Choose a reason for hiding this comment

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

방 상태가 닫힌 경우에 프로필 업데이트 한 번 해주네요
구현하느라 고생했어요 ㅎ.ㅎ 감사합니다!!

ashsty
ashsty approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@ashsty ashsty left a comment

Choose a reason for hiding this comment

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

안녕하세요 뽀로로~ 바뀐 부분 확인했습니다!

당장 크게 수정해야 할 부분은 없을 것 같아 어프루브 드려용.
별개로 update... 메서드명은 수정되어야 할 것 같긴 하네요!

고생하셨습니다!

youngsu5582
youngsu5582 approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@youngsu5582 youngsu5582 left a comment

Choose a reason for hiding this comment

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

굳이용!
조건문 - 업데이트 로직을 누가 가질지는 나중에 좀더 생각해도 괜찮을 거 같아용👍

@jcoding-play jcoding-play merged commit 6a7b8b8 into develop Nov 5, 2024
5 checks passed
@jcoding-play jcoding-play deleted the feat/#711 branch November 5, 2024 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ashsty ashsty ashsty approved these changes

@hjk0761 hjk0761 hjk0761 approved these changes

@youngsu5582 youngsu5582 youngsu5582 approved these changes

Assignees

@jcoding-play jcoding-play

Labels
BE 백엔드 개발 관련 작업 기능 기능 구현 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BE] 방 CLOSE 상태 이후 피드백 개수 증가 로직 수정
4 participants
@ashsty @hjk0761 @youngsu5582 @jcoding-play