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) #23

Merged
merged 5 commits into from
Oct 30, 2023
Merged

게시물 공유(API) #23

merged 5 commits into from
Oct 30, 2023

Conversation

Chestnut90
Copy link
Collaborator

PR 체크리스트

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

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

PR 유형

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

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

현재 동작은 무엇인가요?

이슈 번호: #12

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

  • /shares/<content_id>/ post api 정의 및 테스트

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

  • 예
  • [�x] 아니요

기타 정보

image

@Chestnut90
✅Test : 게시물 공유 api, url 실패 테스트 케이스#12
5da26fb 
- TestCase Red
- /shares/<content_id>/ url에 대한 리소스 찾을 수 없는 에러 발생

Related to #12
@Chestnut90
✨Feat : 게시물 좋아요 api 추가 #12
c3a1a54 
- 게시물 좋아요 api, /shares/<content_id>/ 에 대한 view 인터페이스 추가

Related to #12
@Chestnut90
✅Test : 게시물 좋아요 api에 대한 post 테스트 실패 #12
dc83f45 
- 인증되지 않은 사용자가 게시물에 post 요청을 수행 시 실패 테스트 케이스
- 인증된 사용자가 게시물에 post 요청을 수행 시 실패 테스트 케이스

Related to #12
@Chestnut90
♻️Refact : 게시물 좋아요 api에 대한 기능 완료 #12
77178c9 
- 게시물 좋아요 api에 대한 Post 메시지 요청 시 게시물의 공유수를 1 증가한다.

Related to #12
@Chestnut90 Chestnut90 linked an issue Oct 29, 2023 that may be closed by this pull request
feature: 게시물 공유(API) #12
Closed
@Chestnut90
✅Test : 게시물 공유 api의 no api 테스트 케이스 수정 #12
66ad105 
- flake8에 의해 사용되지 않는 변수 오류가 발생됨.

Related to #12
@Chestnut90 Chestnut90 changed the title Feature/issue 012 feature: 게시물 공유(API) Oct 29, 2023
@Chestnut90 Chestnut90 changed the title feature: 게시물 공유(API) 게시물 공유(API) Oct 29, 2023
@Chestnut90
Copy link
Collaborator Author

Chestnut90 commented Oct 29, 2023 •
edited
Loading

TDD기반으로 작업시 커밋을 작성해 보았습니다.
TDD 기반으로 작업 시 커밋에서도 TDD가 느껴질 수 있는 커밋방법이 있으시면 알려주세요~

참고한 TDD 내용입니다.
https://blog.nerdfactory.ai/2022/09/20/apply-tdd-with-python-django.html

@lfoyh6591
Copy link
Collaborator

LGTM :)

@simseulnyang
Copy link
Collaborator

simseulnyang commented Oct 29, 2023 •
edited
Loading

코드 확인했습니다!
참고하셨다는 TDD관련 블로그 글도 읽어보았는데 TDD에 대해서 또 하나 배우고 가는 것 같아요
좋은 정보 공유해주셔서 감사합니다😊

@JaeHyuckSa
Copy link
Contributor

코드 확인했습니다 :) 고생하셨습니다 ! 타입 적용도 한번 해보시는 건 어떠신가요??

@Chestnut90 Chestnut90 merged commit a3af62f into develop Oct 30, 2023
1 check passed
@JaeHyuckSa JaeHyuckSa deleted the feature/issue-012 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

@lfoyh6591 lfoyh6591 lfoyh6591 approved these changes

@JaeHyuckSa JaeHyuckSa Awaiting requested review from JaeHyuckSa

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 @lfoyh6591 @simseulnyang @JaeHyuckSa