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

[AN] 놀이터 서비스 기능 및 리팩토링 #720

Merged
merged 29 commits into from
Oct 23, 2024
Merged

Conversation

dpcks0509
Copy link
Contributor

이슈

개발 사항

  • 놀이터 서비스 기능 개발
  • 저번 pr 리뷰 반영 및 리팩토링
service.mp4

@dpcks0509 dpcks0509 added 🛠️ refactor 리팩터링 🤖 android android ✨ feat 기능 개발 labels Oct 23, 2024
@dpcks0509 dpcks0509 added this to the Sprint6 milestone Oct 23, 2024
@dpcks0509 dpcks0509 self-assigned this Oct 23, 2024
Copy link

github-actions bot commented Oct 23, 2024

Test Results

60 tests   60 ✅  1s ⏱️
 9 suites   0 💤
 9 files     0 ❌

Results for commit 4734f86.

♻️ This comment has been updated with latest results.

junjange
junjange previously approved these changes Oct 23, 2024
Copy link
Member

@junjange junjange 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 89 to 90
if (getPlaygroundAlarmUseCase.invoke().getOrDefault(true)) {
createNotificationChannel(AlarmType.PLAYGROUND)
Copy link
Member

Choose a reason for hiding this comment

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

getPlaygroundAlarmUseCase() 이렇게도 쓸 수 있어요~~

Comment on lines +39 to +51
is RecentPetNavigationAction.NavigateToOtherProfile -> {
val intent =
OtherProfileActivity.getIntent(
this,
id = navigateAction.memberId,
)
startActivity(intent)
}

is RecentPetNavigationAction.NavigateToPetImage -> {
val intent = PetImageActivity.getIntent(this, navigateAction.petImageUrl)
startActivity(intent)
}
Copy link
Member

Choose a reason for hiding this comment

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

굿

gaeun5744
gaeun5744 previously approved these changes Oct 23, 2024
Copy link
Member

@gaeun5744 gaeun5744 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@dpcks0509 dpcks0509 dismissed stale reviews from gaeun5744 and junjange via 70a1c9f October 23, 2024 12:54
gaeun5744
gaeun5744 previously approved these changes Oct 23, 2024
@dpcks0509 dpcks0509 merged commit 0b9fc31 into develop Oct 23, 2024
2 checks passed
@dpcks0509 dpcks0509 deleted the feature/playground branch October 24, 2024 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android android ✨ feat 기능 개발 🛠️ refactor 리팩터링
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

안드로이드 Service API 대응
4 participants