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

#100 - 이용제한 API의 스펙을 클라이언트의 요구에 맞게 변경한다. #110

Merged
merged 3 commits into from
Aug 28, 2024

Conversation

kpeel5839
Copy link
Contributor

1. 🔗 관련 이슈

2. 📄 구현한 내용 또는 수정한 내용

이번에는 약간 Enum과 Function을 이용하여서, 우선순위를 설정하고, 계산하는 방법을 사용해봤어요.
참고한 글은 다음과 같습니다.(참고한 글)(다르긴한데, 여기서 영감을 얻었어요.)

좋은 패턴인지는 모르겠지만, 재미있는 것 같아서 적용해봤어요.

3. 🙌 추가적으로 알리고 싶은 내용

4. 🙄 TODO / 고민하고 있는 것들

5. ✅ 배포 Checklist

  • 본인을 Assign 해주세요.
  • 본인을 제외한 백엔드 개발자를 리뷰어로 지정해주세요.
  • 변경된 DB 업데이트 해주세요. (꼭 해주세요 배포 안돼요!!!)

@kpeel5839 kpeel5839 requested a review from sectionr0 August 27, 2024 13:41
@kpeel5839 kpeel5839 self-assigned this Aug 27, 2024
@kpeel5839 kpeel5839 added 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요? 🏋️매튜🏋️ 24기 김재연 labels Aug 27, 2024
Copy link
Contributor

@sectionr0 sectionr0 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!👍👍

Comment on lines +30 to +31
{ _ -> true },
{ _ ->
Copy link
Contributor

Choose a reason for hiding this comment

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

오홍 이렇게도 표현할 수 있군요!!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

알러뷰

@github-actions github-actions bot added the 🌟머지 해주세요🌟 머지해주셔! label Aug 28, 2024
@kpeel5839 kpeel5839 merged commit 9964b29 into develop Aug 28, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟머지 해주세요🌟 머지해주셔! 🏋️매튜🏋️ 24기 김재연 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants