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

feat: 태그 메인 페이지 api 요청 #636

Merged
merged 6 commits into from
Aug 9, 2023

Conversation

nyj001012
Copy link
Contributor

@nyj001012 nyj001012 commented Jul 31, 2023

개요

태그 메인 페이지 API 추가

작업 사항

  • 뷰 추가
  • 컨트롤러, 서비스, 레포지토리 함수 추가

@nyj001012 nyj001012 added the enhancement New feature or request label Jul 31, 2023
@nyj001012 nyj001012 self-assigned this Jul 31, 2023
@nyj001012 nyj001012 linked an issue Jul 31, 2023 that may be closed by this pull request
@nyj001012 nyj001012 temporarily deployed to development July 31, 2023 23:51 — with GitHub Actions Inactive
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

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

image

혹시 현재 PR과 연관된 커밋은 첫 두 커밋뿐일까요?

@nyj001012
Copy link
Contributor Author

image

혹시 현재 PR과 연관된 커밋은 첫 두 커밋뿐일까요?

넵 맞아요

@scarf005
Copy link
Member

scarf005 commented Aug 1, 2023

그렇다면 rebase 또는 cherry-pick 기능을 사용해 커밋 둘만 남겨보는 방법을 추천드립니다.

git cherry-pick

git switch origin/develop

# origin/develop을 기반으로 새 브랜치 생성
git switch -c tmp

# 첫 두 커밋을 복사해 tmp 위에 얹습니다
git cherry-pick 0878cad65189dc4893c7dfa2e10273a2f72367a9
git cherry-pick c860b598777b87b849a41a5aee85f1af5898f7d4

# 기존 브랜치 (커밋 x18)를 제거합니다
git branch -D 582-태그-메인-페이지-api-요청

# rebase 된 새 브랜치(커밋 x2) 이름을 기존 브랜치명으로 수정합니다
git switch -c 582-태그-메인-페이지-api-요청

git push -f

참고자료

git rebase --onto <새 base 커밋> <시작 커밋> <끝 커밋>

# 첫 두 커밋을 복사해 origin/develop 위에 얹습니다.
git rebase \
	--onto origin/develop \
	0878cad65189dc4893c7dfa2e10273a2f72367a9^ \
	c860b598777b87b849a41a5aee85f1af5898f7d4

# 기존 브랜치 (커밋 x18)를 제거합니다
git branch -D 582-태그-메인-페이지-api-요청

# rebase 된 새 브랜치(커밋 x2) 이름을 기존 브랜치명으로 수정합니다
git switch -c 582-태그-메인-페이지-api-요청

git push -f

Warning

수정: 마지막 줄에 실수가 있었습니다. rebase시 HEAD 브랜치 이전 커밋만 가져왔기 때문에, 현재 detached HEAD 상태(HEAD가 branch가 아닌 commit 위에 있음)이므로 git switch -c 를 사용하셔야 합니다.

- git branch -m 582-태그-메인-페이지-api-요청
+ git switch -c 582-태그-메인-페이지-api-요청

참고 링크

@nyj001012
Copy link
Contributor Author

오... 지금 봤네요. 이후에 커밋 5개가 추가되었으니 7개 항목에 대해 cherry pick하면 되는 거겠죠?

@scarf005
Copy link
Member

scarf005 commented Aug 1, 2023

넵 7개 항목에 대해 cherry picking을 하는 방식이 가장 간단할 것 같습니다.
git rebase --interactive를 쓰면 더 편리할지도?

@nyj001012
Copy link
Contributor Author

성공했다면 제가 추가한 커밋들만 나와야 할텐데... 계속 시도해보다 점심 때까지 안 되면 dm으로 여쭤봐도 될까요?

@scarf005 scarf005 force-pushed the 582-태그-메인-페이지-api-요청 branch from d6e06fe to 0c51f2a Compare August 1, 2023 02:57
@scarf005 scarf005 temporarily deployed to development August 1, 2023 02:57 — with GitHub Actions Inactive
@nyj001012 nyj001012 marked this pull request as ready for review August 1, 2023 03:00
@scarf005 scarf005 self-assigned this Aug 2, 2023
@scarf005 scarf005 force-pushed the 582-태그-메인-페이지-api-요청 branch from 0c51f2a to 1aa20a7 Compare August 2, 2023 12:56
메인 페이지에서 사용할 super, default 태그 뷰 생성
메인 페이지 용 태그 조회 API 추가
메인 페이지 용 태그 조회 컨트롤러 추가
메인 페이지 용 태그 조회 서비스 추가
메인 페이지 용 태그 조회 레포지토리 함수 추가
@scarf005 scarf005 force-pushed the 582-태그-메인-페이지-api-요청 branch from 1aa20a7 to 70a7a94 Compare August 3, 2023 07:01
view entity expression 구성할 때 createQueryBuilder 대신 raw query 사용
@nyj001012 nyj001012 temporarily deployed to development August 9, 2023 07:03 — with GitHub Actions Inactive
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

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

image

LGTM

@nyj001012 nyj001012 merged commit d5333b7 into develop Aug 9, 2023
3 checks passed
@nyj001012 nyj001012 deleted the 582-태그-메인-페이지-api-요청 branch August 9, 2023 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

태그 메인 페이지 API 요청.
2 participants