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

refactor: 알림 서비스 구조 리팩토링 #235

Merged
merged 6 commits into from
Sep 24, 2022

Conversation

sanbonai06
Copy link
Member

📝 PR Summary

  • 알림에 관한 로직들도 마찬가지로 서비스 구조 리팩토링 진행했습니다.
  • 추가로 자녀의 돈길리스트 중 REJECTED 상태의 돈길은 안보이도록 수정했습니다

🌲 Working Branch

refactor/notificationMapper

🌲 TODOs

Related Issues

#234

@sanbonai06 sanbonai06 added For: API [이슈 대상] 외부 API Type: Refactor [이슈 목적] 프로덕션 코드 리팩토링 labels Sep 24, 2022
@sanbonai06 sanbonai06 added this to the 코드 리팩토링 milestone Sep 24, 2022
@sanbonai06 sanbonai06 requested a review from ozzing September 24, 2022 15:15
@sanbonai06 sanbonai06 self-assigned this Sep 24, 2022
Copy link
Member

@ozzing ozzing left a comment

Choose a reason for hiding this comment

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

허걱 확인이 조금 늦었네요 죄송합니다!! 저도 리팩 달리겠슴다

@ozzing ozzing merged commit 3b9590a into dev Sep 24, 2022
@ozzing ozzing deleted the refactor/notificationMapper branch September 24, 2022 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For: API [이슈 대상] 외부 API Type: Refactor [이슈 목적] 프로덕션 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants