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

#138: 로그인후 이용제한 확인하기 #144

Merged
merged 8 commits into from
Aug 28, 2024

Conversation

flash159483
Copy link
Contributor

1. 📄 관련된 이슈 및 소개

close #138

2. 🔥 변경된 점

이용제한을 확인하는 로직이 추가됐어요

3. ✅ 필수 체크 사항

  • 이용제한 확인
  • 관련 Bottom Sheet 표시

4. 📸 작업물 사진 공유(선택)

untitled.webm

5. 💡알게된 혹은 궁금한 사항

@flash159483 flash159483 added 🌱기능🌱 새로운 기능을 추가해요 ! 🍩브라우니🍩 24기 정승원 labels Aug 27, 2024
@flash159483 flash159483 requested a review from jeongjaino August 27, 2024 14:30
@flash159483 flash159483 self-assigned this Aug 27, 2024
…m-1-Android into feature/flash159483/#138

# Conflicts:
#	app/src/main/kotlin/com/bff/wespot/MainActivity.kt
#	app/src/main/kotlin/com/bff/wespot/viewmodel/MainViewModel.kt
@flash159483 flash159483 reopened this Aug 27, 2024
Comment on lines +453 to +605
text = stringResource(com.bff.wespot.auth.R.string.confirm),
paddingValues = PaddingValues(
start = 0.dp,
end = 0.dp,
top = 24.dp,
bottom = 10.dp
),
) {
it.invoke()
}
} else {
Row(
modifier = Modifier
.fillMaxWidth()
.padding(top = 20.dp),
horizontalArrangement = Arrangement.spacedBy(10.dp),
) {
Box(modifier = Modifier.weight(1f)) {
WSButton(
onClick = {
coroutineScope.launch {
bottomSheetState.hide()
}
},
buttonType = WSButtonType.Secondary,
text = stringResource(R.string.close),
paddingValues = PaddingValues(0.dp),
) {
it()
}
}

Box(modifier = Modifier.weight(1f)) {
WSButton(
onClick = {
navigator.navigateToWebLink(context, state.kakaoChannel)
},
text = stringResource(com.bff.wespot.R.string.one_on_one),
paddingValues = PaddingValues(0.dp),
) {
it()
}
}
}
}
}
}
}

@Composable
private fun BulletPoint(text: String) {
Row(
horizontalArrangement = Arrangement.spacedBy(6.dp)
) {
Text(
text = "•",
style = StaticTypeScale.Default.body6,
color = WeSpotThemeManager.colors.txtSubColor,
)

Text(
text = text,
style = StaticTypeScale.Default.body6,
color = WeSpotThemeManager.colors.txtSubColor,
)
}
}

private enum class RestrictionContent(
@StringRes val title: Int,
@StringRes val body1: Int,
@StringRes val body2: Int,
@StringRes val body3: Int,
@StringRes val body4: Int? = null,
val buttonNumber: Int = 1,
) {
TYPE1(
com.bff.wespot.R.string.restriction_title,
com.bff.wespot.R.string.restriction_type1_body1,
com.bff.wespot.R.string.restriction_type1_body2,
com.bff.wespot.R.string.restriction_type1_body3,
com.bff.wespot.R.string.restriction_type1_body4,
),

TYPE2(
com.bff.wespot.R.string.restriction_title,
com.bff.wespot.R.string.restriction_type1_body1,
com.bff.wespot.R.string.restriction_type2_body2,
com.bff.wespot.R.string.restriction_type2_body3,
),

TYPE3(
com.bff.wespot.R.string.restriction_title,
com.bff.wespot.R.string.restriction_type1_body1,
com.bff.wespot.R.string.restriction_type3_body2,
com.bff.wespot.R.string.restriction_type3_body3,
null,
2,
);
}
Copy link
Member

Choose a reason for hiding this comment

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

MainActivity가 계속계속 커지는 느낌이라, 따로 앱 모듈 내에 component 패키지 만들어서 컴포저블 함수 옮기는 건 어떨까요 ??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

귀찮으니 다음에 해야겠다

Comment on lines 141 to 147
LazyColumn {
items(pagingData.itemCount, key = { index ->
pagingData[index]?.id ?: index
}) { index ->
items(
pagingData.itemCount,
key = pagingData.itemKey { it.id },
) { index ->
val school = pagingData[index]

Copy link
Member

Choose a reason for hiding this comment

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

좋아영~ 저도 다 바꾸겠습니당~~

@github-actions github-actions bot added the 🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. label Aug 28, 2024
@flash159483 flash159483 merged commit 4938ffb into develop Aug 28, 2024
7 checks passed
@flash159483 flash159483 deleted the feature/flash159483/#138 branch August 28, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟머지 해주세요🌟 코드 리뷰가 완료된 뒤 PR을 올린사람이 Merge를 하면 되는 단계입니다. 🌱기능🌱 새로운 기능을 추가해요 ! 🍩브라우니🍩 24기 정승원
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WS-412 [FEATURE]: 로그인 후 이용제한 확인
2 participants