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

♻️ user block logic #437

Open
wants to merge 3 commits into
base: be/dev
Choose a base branch
from
Open

♻️ user block logic #437

wants to merge 3 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 11, 2024

차단로직을 수정했습니다. 기존 단방향 차단만 존재했다면 이제 양방향으로 차단되도록 했습니다.

  • 기존 : 내가 차단하는 사람의 컨텐츠가 안보임
  • 변경 후 : 내가 차단하는 사람 + 나를 차단하는 사람의 컨텐츠 모두 보이지 않음

closes #436

@github-actions github-actions bot added ✨ feature new feature BE Backend ♻️ refactor refactor labels Dec 11, 2024
@hyxrxn hyxrxn changed the title ♻️ user blcok logic ♻️ user block logic Dec 15, 2024
Copy link
Contributor Author

github-actions bot commented Jan 6, 2025

Overall Project 89.12% -1.66% 🍏
Files changed 49.02% 🍏

File Coverage
BlockeeGroup.java 100% 🍏
UserService.java 93.61% 🍏
BlockerGroup.java 76.47% -23.53% 🍏
BlockedUserFilterAspect.java 42.29% -39.43% 🍏
ForbiddenException.java 0% 🍏

@HaiSeong
Copy link

HaiSeong commented Jan 6, 2025

차단로직을 수정했습니다. 기존 단방향 차단만 존재했다면 이제 양방향으로 차단되도록 했습니다.

  • 기존 : 내가 차단하는 사람의 컨텐츠가 안보임
  • 변경 후 : 내가 차단하는 사람 + 나를 차단하는 사람의 컨텐츠 모두 보이지 않음

@HaiSeong HaiSeong requested review from HaiSeong, hyxrxn and tackyu January 6, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE Backend ♻️ refactor refactor ✨ feature new feature
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant