feat: 앱 문의 채널 변경 #737
Merged
feat: 앱 문의 채널 변경 #737
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Note Reviews pausedUse the following commands to manage reviews:
Walkthrough앱 문의 채널의 URL을 관리하는 string 리소스( Changes
Sequence Diagram(s)(해당 변경은 단순 리소스 값 및 참조 교체와 빌드 설정 변경으로, 시퀀스 다이어그램 생성이 필요하지 않습니다.) Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Assessment against linked issues
Assessment against linked issues: Out-of-scope changes
Suggested reviewers
Poem
Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
✅ Files skipped from review due to trivial changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
yeonjeen
approved these changes
Aug 9, 2025
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🔭 Outside diff range comments (1)
app/src/main/java/com/into/websoso/ui/createFeed/CreateFeedSearchNovelBottomSheetDialog.kt (1)
131-138: 외부 브라우저 호출 시 이중 클릭 방지 및 안전성 보강
- 이중 탭으로 여러 브라우저가 열릴 수 있습니다. 기존 패턴과 동일하게
singleEventHandler.throttleFirst적용을 권장합니다.resolveActivity로 처리 가능 앱 존재 여부를 확인해 크래시를 방지하세요.private fun setupNavigateToInquireNovel() { tracker.trackEvent("contact_novel_connect") - val inquireUrl = getString(novel_inquire_link) + val inquireUrl = getString(novel_inquire_link) val intent = Intent(Intent.ACTION_VIEW, Uri.parse(inquireUrl)) binding.tvCreateFeedAddNovelInquireButton.setOnClickListener { - startActivity(intent) + singleEventHandler.throttleFirst { + val pm = requireContext().packageManager + if (intent.resolveActivity(pm) != null) { + startActivity(intent) + } else { + // TODO: 사용자 피드백 (Toast/Snackbar) 고려 + } + } } }추가로, 클릭 시점에도 별도 트래킹 이벤트를 남기면(예:
contact_novel_connect_click) 퍼널 분석에 도움이 됩니다.
🧹 Nitpick comments (2)
core/resource/src/main/res/values/deepLinks.xml (1)
3-4: URL 리소스에 translatable="false" 지정 권장고정 URL은 현지화 대상이 아니므로 실수로 번역 큐에 올라가지 않도록
translatable="false"를 추가하는 것을 권장합니다.- <string name="inquire_link">https://www.notion.so/helpwebsoso/241a9688d1a381548c20dd314d0a0b0a?pvs=106</string> - <string name="novel_inquire_link">https://helpwebsoso.notion.site/241a9688d1a38164b3f8efd0b51edaab</string> + <string name="inquire_link" translatable="false">https://www.notion.so/helpwebsoso/241a9688d1a381548c20dd314d0a0b0a?pvs=106</string> + <string name="novel_inquire_link" translatable="false">https://helpwebsoso.notion.site/241a9688d1a38164b3f8efd0b51edaab</string>app/src/main/java/com/into/websoso/ui/createFeed/CreateFeedSearchNovelBottomSheetDialog.kt (1)
15-15: 리소스 ID 개별 import는 지양 권장(선택 사항)팀 컨벤션에 따라 다르지만, 개별
R.string.foo상수 import 대신R만 import 후R.string.novel_inquire_link로 참조하면 가독성과 일관성이 좋아집니다.- import com.into.websoso.core.resource.R.string.novel_inquire_link + import com.into.websoso.core.resource.R ... - val inquireUrl = getString(novel_inquire_link) + val inquireUrl = getString(R.string.novel_inquire_link)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
app/src/main/java/com/into/websoso/ui/createFeed/CreateFeedSearchNovelBottomSheetDialog.kt(2 hunks)core/resource/src/main/res/values/deepLinks.xml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
app/src/main/java/com/into/websoso/ui/createFeed/CreateFeedSearchNovelBottomSheetDialog.kt
Show resolved
Hide resolved
junseo511
approved these changes
Aug 9, 2025
s9hn
approved these changes
Aug 9, 2025
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
core/resource/build.gradle.kts (2)
9-11: PR 스코프 일탈 가능성 — 플랫폼 전략 변경은 별도 PR로 분리 권장문의 채널 리소스/텍스트 교체가 주 목적이라면, minSdk 상향은 독립된 PR로 분리하여 QA/릴리즈 노트를 명확히 하는 것이 안전합니다. 영향 범위(지원 OS 버전, 스토어 타겟 단말, 분석 대시보드 단절 구간 등) 파악과 롤백 용이성을 위해 분리 제안드립니다.
9-11: 구성 일관성/유지보수성 개선 제안: minSdk 중앙집중화여러 모듈에서 minSdk를 개별 하드코딩하면 드리프트가 발생하기 쉽습니다. 버전 카탈로그나 공통 Gradle convention 플러그인에서 단일 소스로 관리하도록 리팩터링을 권장합니다.
예시(버전 카탈로그 사용 시):
- defaultConfig { - minSdk = 30 - } + defaultConfig { + // libs.versions.toml에 [versions] minSdk = "30" 정의 후 사용 + minSdk = libs.versions.minSdk.get().toInt() + }또는 사내 build-logic(공통 android-library 설정)에서 일괄 적용하도록 이전을 검토해주세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
core/resource/build.gradle.kts(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (1)
core/resource/build.gradle.kts (1)
9-11: minSdk 30 상향 의도 확인 요청
- core/resource/build.gradle.kts (9–11) defaultConfig.minSdk = 30
- build-logic/src/main/kotlin/websoso.android.kotlin.gradle.kts (14) minSdk = 30
라이브러리 모듈의 minSdk를 30으로 올리면, 이 라이브러리를 사용하는 앱 모듈의 minSdk가 30 미만일 경우 지원 기기가 Android 10(API 29) 이하에서 제외되거나 빌드/머지 시 오류가 발생할 수 있습니다.
이번 PR의 주 목적인 “문의 채널 변경”과 직접적인 관련이 적어 보여, 정책·운영적 변경인지 재확인 부탁드립니다.앱 모듈들의 minSdk 설정을 검토하시고, 문제가 없다면 승인 여부를 알려주세요.
전체 모듈의 minSdk 설정 분포는 아래 스크립트로 확인 가능합니다:#!/bin/bash set -euo pipefail echo "== minSdk occurrences across the repo ==" rg -n --glob '!**/build/**' -H -e 'minSdk\s*=\s*\d+' -e 'minSdkVersion\s*"?\d+"?' || true
Member
Author
|
@coderabbitai pause |
✅ Actions performedReviews paused. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
문의하기 링크로 연결되어 있어서작품 문의하기 링크로 수정했습니다.📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴
Summary by CodeRabbit