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

게시물 좋아요 API 작성 #17

Merged
merged 10 commits into from
Oct 27, 2023
Merged

게시물 좋아요 API 작성 #17

merged 10 commits into from
Oct 27, 2023

Conversation

Chestnut90
Copy link
Collaborator

@Chestnut90 Chestnut90 commented Oct 26, 2023 •
edited
Loading

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

현재 동작은 무엇인가요?

이슈 번호: #11

새로운 동작은 무엇인가요?

  • 게시물(post)에 대한 좋아요(likes) api 추가 및 테스트

이 PR은 호환성 변경을 도입하나요?

  • 예
  • 아니요

기타 정보

@Chestnut90
✨Feat : 게시물 좋아요 api #11
c521486 
- 게시물 좋아요, /likes/<content_id>/ 의 post method에 대한 APIView
Related to #11
@Chestnut90
✅Test : 게시물 좋아요 api 테스트 #11
f10bf1c 
- authenticated 되지 않은 경우Unauthorized(401)
- authenticated 된 경우 like_count를 증가하며 Ok(200)

Related to #11
@Chestnut90 Chestnut90 linked an issue Oct 26, 2023 that may be closed by this pull request
feature: 게시물 좋아요(API) #11
Closed
@JaeHyuckSa JaeHyuckSa changed the title Feature/issue 011 게시물 좋아요(API) Oct 26, 2023
@JaeHyuckSa JaeHyuckSa changed the title 게시물 좋아요(API) 게시물 좋아요 API 작성 Oct 26, 2023
@JaeHyuckSa JaeHyuckSa added the enhancement New feature or request label Oct 26, 2023
@JaeHyuckSa
Copy link
Contributor

코드 작성하시느라 고생하셨습니다 ! :)

@lfoyh6591
Copy link
Collaborator

post_type에 따른 url 분리가 구현이 안된 것 같습니다.
image

@simseulnyang
Copy link
Collaborator

코드 확인했습니다. 수고하셨습니다!

@JaeHyuckSa
Copy link
Contributor

JaeHyuckSa commented Oct 26, 2023 •
edited by Chestnut90
Loading

추가적인 내용 (회의 정리 내용)

  • Post Model에 content_id 추가

@Chestnut90
♻️Refact : Post 모델 리팩토링 #4
de812ca 
- content_id 필드 추가, social post에 대한 구분 식별자

Related to #4
@Chestnut90
♻️Refact : 좋아요 api에 대한 content_id 적용
5758afc 
- post 모델의 content_id 필드가 추가됨에 따라서 likes/<content_id>/ 리소스 형태로 변경

Related to #11
@Chestnut90
Copy link
Collaborator Author

Post 모델에 대해 content_id를 uuid로 기본 설정하였습니다.
Serializer도 사용하는 field만 적용하도록 변경하였습니다.

JaeHyuckSa
JaeHyuckSa reviewed Oct 26, 2023
View reviewed changes
Copy link
Contributor

@JaeHyuckSa JaeHyuckSa left a comment

Choose a reason for hiding this comment

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

코드 리뷰

likes/views.py Outdated Show resolved Hide resolved
config/urls.py Outdated Show resolved Hide resolved
likes/tests.py Show resolved Hide resolved
likes/urls.py Outdated Show resolved Hide resolved
likes/views.py Outdated Show resolved Hide resolved
posts/models.py Show resolved Hide resolved
@JaeHyuckSa JaeHyuckSa added feature and removed enhancement New feature or request labels Oct 26, 2023
@Chestnut90
♻️Refact : LikeApi의 post object getter 변경
646f084 
- post 모델에 대한 getter는 django의 내장 함수로 대체 가능
- 가독성 증가

Related to #11
@Chestnut90
♻️Refact : api url 포맷 일치 #11
bd40d3d 
- 해당 프로젝트의 url format을 api/{application}/ 로 사용하기로 정함.

Related to #11
@Chestnut90
♻️Refact : API name 부여 및 testcase 리팩토링 #11
5c2632e 
- 뷰에 대한 이름 지정
- API 테스트에서 url을 reverse를 통해 가져올 수 있도록
- 추후 뷰에 대한 변경이 파급이 적도록

Related to #11
@Chestnut90
♻️Refact : 절대경로 import 변경 #11
88ac02b 
- PEP8 권장에 따른 상대경로에서 절대경로로 변경함

Related to #11
@Chestnut90
♻️Refact : likes api에 대한 swagger 적용 #11
e7ed99b 
- api 문서화를 위한 swagger 적용

Related to #11
@JaeHyuckSa JaeHyuckSa self-requested a review October 27, 2023 01:46
@Chestnut90 Chestnut90 merged commit cdfb8b9 into develop Oct 27, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/issue-011 branch November 19, 2023 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simseulnyang simseulnyang simseulnyang approved these changes

@JaeHyuckSa JaeHyuckSa JaeHyuckSa approved these changes

@lfoyh6591 lfoyh6591 Awaiting requested review from lfoyh6591

Assignees

@Chestnut90 Chestnut90

Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feature: 게시물 좋아요(API)
4 participants
@Chestnut90 @JaeHyuckSa @lfoyh6591 @simseulnyang