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] feat: 강아지 선택 기능 구현 #90

Merged
merged 10 commits into from
Jul 18, 2024

Conversation

jinuemong
Copy link
Contributor

@jinuemong jinuemong commented Jul 17, 2024

이슈

개발 사항

  • 강아지 선택 기능 구현하였습니다.
Screen_recording_20240717_150555.mp4

@jinuemong jinuemong added 🤖 android android ✨ feat 기능 개발 labels Jul 17, 2024
@jinuemong jinuemong self-assigned this Jul 17, 2024
Copy link

github-actions bot commented Jul 17, 2024

Test Results

3 tests   3 ✅  0s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit e84afd2.

♻️ This comment has been updated with latest results.

junjange
junjange previously approved these changes Jul 17, 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.

고생하셨습니다~~👍

gaeun5744
gaeun5744 previously approved these changes Jul 17, 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.

수고많았습니다!!

Comment on lines 5 to 9
data class DogSelectUiModel(
val id: Long,
val name: String,
val profileImage: String,
) : Serializable {
Copy link
Member

Choose a reason for hiding this comment

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

혹시 @serializable로 바꿀 수 있을까요? import java에서 kotlin으로 하는게 안정성 면에서 좋을거 같아서요..!

@J-I-H-O
Copy link
Contributor

J-I-H-O commented Jul 17, 2024

머싯당

@jinuemong jinuemong dismissed stale reviews from gaeun5744 and junjange via e84afd2 July 18, 2024 01:40
@jinuemong jinuemong requested review from junjange and gaeun5744 July 18, 2024 01:45
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.

수고하셨습니당~

Copy link
Contributor

@dpcks0509 dpcks0509 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 +26 to +35
private fun loadMyDogs() =
viewModelScope.launch {
delay(1000)
_dogs.value =
listOf(
DogSelectUiModel(
id = 0L,
profileImage = "",
name = "강아지 1",
),
Copy link
Contributor

Choose a reason for hiding this comment

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

delay를 따로 준 이유가 있나요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

실제 데이터 로드에 대한 대비를하기 위해서입니다!
추후에 상태 관리에 대한 코드를 위해 작성하였습니다.

@jinuemong jinuemong merged commit ffcb405 into develop Jul 18, 2024
2 checks passed
@jinuemong jinuemong deleted the feature/group-select-dog branch July 25, 2024 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android android ✨ feat 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

강아지 선택 기능 구현
5 participants