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

[#286] 홈화면 태그 필터 추가 #293

Merged
merged 19 commits into from
Sep 10, 2024
Merged

[#286] 홈화면 태그 필터 추가 #293

merged 19 commits into from
Sep 10, 2024

Conversation

JeonK1
Copy link
Collaborator

@JeonK1 JeonK1 commented Sep 2, 2024

Issue No

Overview (Required)

  • 홈화면 태그 필터 추가
  • PicTag 의 background 와 padding 을 유연하게 수정할 수 있도록 backgroundColor 에 대한 내용을 Modifier 에서 수정할 수 있도록 변경

Links

Screenshot

_2024_09_03_22_50_20_434.mp4

@JeonK1 JeonK1 self-assigned this Sep 2, 2024
@JeonK1 JeonK1 marked this pull request as draft September 2, 2024 13:39
@JeonK1 JeonK1 marked this pull request as ready for review September 2, 2024 14:06
Copy link
Collaborator

@sooziini sooziini left a comment

Choose a reason for hiding this comment

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

고생했어~ 오빠 열일하네 요즘.
한 가지 궁금한 게 나는 태그에 해당하는 그룹이 없는 경우에는 그 태그를 안 보여주는 걸로 이해했었거둔 그래서 figma에 코멘트 남겨두었엉. 뭔가 모든 태그를 보여주게되면 해당하는 그룹이 없을 때는 empty 표시를 해준다는 등 뭔가 추가 처리가 필요할 거 같아서 말이지

Copy link
Collaborator

@SEO-J17 SEO-J17 left a comment

Choose a reason for hiding this comment

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

계원아 고생했어~~~ 늦은 리뷰 먄먄 ㅠㅠ 코멘트 몇개 달아놨어..! 요즘 작업 너무 열심히 하는거 아니야~? ㅋㄷㅋㄷ 멋져..!

import java.net.UnknownHostException
import javax.inject.Inject

@HiltViewModel
class GroupHomeViewModel @Inject constructor(
getGroupListUseCase: GetGroupListUseCase,
) : ViewModel() {
private val groupUiState = getGroupListUseCase()
private val filterTagUiState: MutableStateFlow<List<FilterTagUiModel>> = MutableStateFlow(
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 stateFlow로 따로 뺀 이유가 있엉?? groupUiState 안에 인자 추가로 처리해줄 수 있을거 같은데 따로 분리해서 combine한 이유가 있을까??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

엇 혹시 인자 추가 처리가 어떤건지 구체적으로 알 수 있을까..?!

Copy link
Collaborator

Choose a reason for hiding this comment

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

늦은 리뷰 미안..ㅎ.ㅎ 나 저거 dataclaas로 한줄 알았어..ㅎ 그리고 저 uiState가 flow였군..! ㅎ.. 잘못봤슴다 제가

Copy link
Member

@oreocube oreocube 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
Collaborator

@SEO-J17 SEO-J17 left a comment

Choose a reason for hiding this comment

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

아이 늦게 어푸해서 죄삼다~~~ 어푸 할게용! 계원이 고생했어~~

@sooziini
Copy link
Collaborator

한 가지 궁금한 게 나는 태그에 해당하는 그룹이 없는 경우에는 그 태그를 안 보여주는 걸로 이해했었거둔 그래서 figma에 코멘트 남겨두었엉. 뭔가 모든 태그를 보여주게되면 해당하는 그룹이 없을 때는 empty 표시를 해준다는 등 뭔가 추가 처리가 필요할 거 같아서 말이지

저 부분 코멘트 달렸는데 태그에 해당하는 그룹이 없을 때는 그 태그를 안 보여주게끔 해야 한다더라고! 일단 이 pr은 머지하고 이슈 따서 추가 작업하는걸루.

@JeonK1
Copy link
Collaborator Author

JeonK1 commented Sep 10, 2024

한 가지 궁금한 게 나는 태그에 해당하는 그룹이 없는 경우에는 그 태그를 안 보여주는 걸로 이해했었거둔 그래서 figma에 코멘트 남겨두었엉. 뭔가 모든 태그를 보여주게되면 해당하는 그룹이 없을 때는 empty 표시를 해준다는 등 뭔가 추가 처리가 필요할 거 같아서 말이지

저 부분 코멘트 달렸는데 태그에 해당하는 그룹이 없을 때는 그 태그를 안 보여주게끔 해야 한다더라고! 일단 이 pr은 머지하고 이슈 따서 추가 작업하는걸루.

@sooziini 대박... 물어봐줘서 고마워!! 이슈파서 따로 작업할게~~

@JeonK1 JeonK1 merged commit e463c7e into develop Sep 10, 2024
1 check passed
@JeonK1 JeonK1 deleted the feature/#286 branch September 20, 2024 10:24
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.

그룹 홈 화면에서 태그별로 필터링하여 보여주기
4 participants