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

feature 팀 정보 조회 API 이용하여 team 이름, profilePath 이용 되도록 수정 #78

Merged
merged 11 commits into from
Jun 21, 2022

Conversation

onemask
Copy link
Collaborator

@onemask onemask commented Jun 20, 2022

스크린샷 2022-06-21 오전 1 01 10

@onemask onemask requested a review from wjdwns June 20, 2022 16:03
@onemask onemask self-assigned this Jun 20, 2022
Copy link
Collaborator

@wjdwns wjdwns left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 저도 얼른 멤버업데이트까지 api 붙여서 mainFragment까지 문제없이 갈수있도록 작업해두겠습니다~~!!

@@ -117,7 +117,7 @@ class MainFragment : Fragment() {
})
binding.rvWeek.adapter = dayOfAdapter

val list = mainViewModel.houseWorks.value?.houseWorks?.toMutableList() ?: mutableListOf()
val list = mainViewModel.myHouseWorks.value?.houseWorks?.toMutableList() ?: mutableListOf()
Copy link
Collaborator

Choose a reason for hiding this comment

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

이제 다른 멤버의 집안일도 들어오니까 리네임하셨나보군요 센스쟁이..

@@ -36,6 +37,9 @@ class GroupProfileAdapter(
RecyclerView.ViewHolder(binding.root) {
fun bind(assign: Assignee) {
binding.assignTemp = assign
Glide.with(binding.root)
.load(assign.profilePath)
.into(binding.ivIcon)
Copy link
Collaborator

Choose a reason for hiding this comment

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

작업해주셔서 감사합니다 ^^7 피곤하실텐데 수고하셨습니다!

@wjdwns
Copy link
Collaborator

wjdwns commented Jun 21, 2022

저도 이제 유저 정보 기입받는 것 전체적으로 마무리하고 메인화면으로 잘 연결될 수 있게끔 해보겠습니다.

@onemask onemask merged commit a9d9263 into develop Jun 21, 2022
@onemask onemask deleted the feature/main branch June 21, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants