Skip to content

Feature: Add SponsorLevelViewSet #19

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

Merged
merged 6 commits into from
Feb 12, 2023

Conversation

oleveloper
Copy link
Member

@oleveloper oleveloper commented Feb 11, 2023

목표

  • Sponsor 등급 정보를 조회할 수 있다
  • 남은 구좌 개수를 조회할 수 있다

작업내용

  • Router 추가
  • 후원사 등급 정보를 조회하기 위한 ViewSet 추가
  • 남은 구좌 개수를 조회하기 위한 ViewSet 추가
  • 남은 구좌 개수를 조회하기 위한 Field(remaining) 추가

유의사항

  • 리뷰어는 PyConKR-2023을 지정 해 주세요.
  • 작업한 내용을 상세하게 작성해주세요.

@oleveloper oleveloper requested a review from a team February 11, 2023 13:11
]

def get_remaining(self, obj):
return obj.current_remaining_number
Copy link
Member

Choose a reason for hiding this comment

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

레거시 모델에 답이 있었네요. :-)

감사합니다.

 - Prospectus API does not show the remaining accounts
@golony6449
Copy link
Member

감사합니다.

아래의 노션 문서 참고하셔서, isort, black linter만 적용해주시면 될 것 같아요 :-)
https://www.notion.so/pyconkorea/PR-69faeb778d43407cbfaa17e6986af15f?pvs=4

@oleveloper
Copy link
Member Author

감사합니다.

아래의 노션 문서 참고하셔서, isort, black linter만 적용해주시면 될 것 같아요 :-) https://www.notion.so/pyconkorea/PR-69faeb778d43407cbfaa17e6986af15f?pvs=4

isort, black을 적용해서 push 해본 결과 서로 code formatting 에 충돌이 있는 것 같아 보여요.
이 부분은 따로 확인이 필요할 것 같습니다.

@golony6449
Copy link
Member

isort, black을 적용해서 push 해본 결과 서로 code formatting 에 충돌이 있는 것 같아 보여요. 이 부분은 따로 확인이 필요할 것 같습니다.

@oleveloper
앗 그렇네요.
blackisort가 import를 처리하는게 다르네요.

일단은 black 기준으로 처리해주시면 될 듯 합니다.

  • 배포 스크립트는 black 포멧팅만을 검증합니다.

@golony6449
Copy link
Member

image
해당 내용은 노션 문서에도 반영해두었습니다 :-)

@golony6449 golony6449 merged commit 52edb2b into pythonkr:devdev Feb 12, 2023
@golony6449
Copy link
Member

@oleveloper 이어서 머지할게 있어서, 제가 사라락 머지했습니다 ㅠㅠ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants