Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PC-596] 설정 페이지 웹뷰 및 API 연결 #67

Merged
merged 10 commits into from
Feb 14, 2025

Conversation

tgyuuAn
Copy link
Member

@tgyuuAn tgyuuAn commented Feb 13, 2025

1. ⭐️ 변경된 내용

  • 설정 페이지 웹뷰 연결
  • 설정 페이지 API 연결

2. 🖼️ 스크린샷(선택)

2025-02-13.6.12.55.mov

3. 💡 알게된 부분

  • 웹뷰로 노션 뛰우는 방법

위 사진 처럼 웹뷰 페이지가 뜨지 않을 때,

웹뷰 속성 아래에 아래 속성을 넣으면 해결 됨!!

layoutParams = ViewGroup.LayoutParams(
    ViewGroup.LayoutParams.MATCH_PARENT,
    ViewGroup.LayoutParams.MATCH_PARENT,
)

@tgyuuAn tgyuuAn added UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 기능 ⚒️ 새로운 기능 구현 ⚒️ 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn labels Feb 13, 2025
@tgyuuAn tgyuuAn requested a review from sksowk156 February 13, 2025 08:33
@tgyuuAn tgyuuAn self-assigned this Feb 13, 2025
Comment on lines +31 to +32
@Serializable
data class WebViewRoute(val title: String, val url: String) : SettingGraphDest()
Copy link
Member Author

Choose a reason for hiding this comment

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

이 화면에서는 네비게이션 바가 없어져야 해서 추가했습니다!

Comment on lines +15 to +26
@Composable
internal fun WebViewRoute(
title: String,
url: String,
viewModel: WebViewViewModel = hiltViewModel(),
) {
WebViewScreen(
title = title,
url = url,
onBackClick = { viewModel.navigationHelper.navigate(NavigationEvent.NavigateUp) },
)
}
Copy link
Member Author

Choose a reason for hiding this comment

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

웹뷰 화면에서는 State도 없고 MVI를 넣을필요가 전혀 없을 것 같아서 그냥 MavericksViewModel이 아니라 ViewModel Inject 시켰어용~~

Copy link
Contributor

@sksowk156 sksowk156 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!! 👍 👍

@sksowk156 sksowk156 added 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟 and removed 리뷰 원해요🔥 피어의 리뷰를 기다리는 ing.. 🔥 labels Feb 13, 2025
@tgyuuAn tgyuuAn merged commit e85cded into develop Feb 14, 2025
@tgyuuAn tgyuuAn deleted the feature/tgyuu/PC-596 branch February 14, 2025 05:25
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
UI/UX 🎨 디자인 시스템, 디자인 리소스 관련 코드 🎨 ㅌㄱ태규 ☀️ 훗날 크게될 ENFP 남성, tgyuuAn 기능 ⚒️ 새로운 기능 구현 ⚒️ 머지 해도될듯염🌟 현재 코드를 기존 코드에 합쳐도 될 것 같다라고 판단..! 🌟
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants