Skip to content

Fix/#443: 태그 조회 수정 #444

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
Merged
hyunn522 merged 5 commits into from
Apr 4, 2025
Merged

Fix/#443: 태그 조회 수정 #444

hyunn522 merged 5 commits into from
Apr 4, 2025

Conversation

@hyunn522
Copy link
Member

@hyunn522 hyunn522 commented Apr 3, 2025 •
edited
Loading

📝 요약

🔖 변경 사항

  • 하나의 게시판에 태그가 여러 개가 있는 경우에 대한 처리 추가
  • 게시판 아이디 + 태그 아이디로 게시물 조회 시, 해당 게시물의 태그 정보 포함하여 조회
  • 구독한 게시판 조회 시 태그 아이디 추가

✅ 리뷰 요구사항

📸 확인 방법 (선택)

image 로컬에서 테스트 완료했습니다.

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

@hyunn522 hyunn522 self-assigned this Apr 3, 2025
@hyunn522 hyunn522 linked an issue Apr 3, 2025 that may be closed by this pull request
Fix: 태그 조회 수정 #443
Closed
@hyunn522 hyunn522 changed the title Fix/#443/태그 조회 수정 Fix/#443: 태그 조회 수정 Apr 3, 2025
seongjunnoh
seongjunnoh approved these changes Apr 3, 2025
View reviewed changes
src/main/java/space/space_spring/domain/home/application/service/ReadHomeService.java
if (tagId != null) {
Tag tag = loadTagPort.loadById(subscription.getTagId());
tagName = tag.getTagName();
latestPost = loadPostPort.loadLatestPostByBoardIdAndTagId(board.getId(), tag.getId());
Copy link
Collaborator

@seongjunnoh seongjunnoh Apr 3, 2025

Choose a reason for hiding this comment

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

LGTM

@hyunn522 hyunn522 merged commit a632c9d into develop Apr 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seongjunnoh seongjunnoh seongjunnoh approved these changes

Assignees

@hyunn522 hyunn522

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix: 태그 조회 수정

3 participants

@hyunn522 @seongjunnoh