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

Feat/#499 알림메시지 변경하기 2 #531

Open
wants to merge 24 commits into
base: dev
Choose a base branch
from

Conversation

BackFoxx
Copy link
Collaborator

@BackFoxx BackFoxx commented Nov 5, 2023

🔥 연관 이슈

📝 작업 요약

처음 올렸던 알림메시지 변경하기 PR에서 받은 피드백을 반영한 새 PR이에요!

🔎 작업 상세 설명

머지를 잘못해서 커밋 기록이 심하게 꼬이고 테스트도 자꾸 깨져서,
dev로부터 브랜치를 새로 만들고 제 작업 기록을 cheery-pick하는 방식으로 복구했어요!
피드백을 받은 이후에 작업한 내용을 쉽게 알아볼 수 있게 피드백 별로 커밋을 남겨두었으니,
커밋 별로 하나씩 보시면 편하실 거에요 :)

image

커밋 1 - 캐시 관리소 만들기
커밋 2 - 관리소 사용하기

image

커밋

image

커밋

image

커밋

🌟 리뷰 요구 사항

테스트가 다소 부족해서, 제대로 동작하는지는 안드로이드로부터 앱을 받아서 QA를 해야할 것 같아요!

@BackFoxx BackFoxx added the ❤️ Backend for BackEnd label Nov 5, 2023
@BackFoxx BackFoxx self-assigned this Nov 5, 2023
Copy link

github-actions bot commented Nov 5, 2023

Unit Test Results

  73 files  +  3    73 suites  +3   24s ⏱️ -2s
380 tests +11  378 ✔️ +11  2 💤 ±0  0 ±0 
383 runs  +11  381 ✔️ +11  2 💤 ±0  0 ±0 

Results for commit 24279ec. ± Comparison against base commit 446640f.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@hectick hectick left a comment

Choose a reason for hiding this comment

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

고생하셨어요 여우씌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
❤️ Backend for BackEnd
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

알림 메시지를 설정할 수 있는 API 만들기
2 participants