docs. Remote Config 필수 키 안내 추가#20
Open
swift-man wants to merge 1 commit into
Open
Conversation
강제 업데이트, 차단 버전, 선택 업데이트, 공지에 필요한 Firebase Remote Config 키를 기능별 표로 정리했습니다. 강제 업데이트와 차단 버전이 동일한 이동 URL 키를 공유한다는 점도 명시했습니다.
There was a problem hiding this comment.
Firebase Remote Config 필수 키에 대한 표 및 세부 설명이 README.md에 추가되어 사용자의 연동 편의성이 대폭 향상되었습니다. 커스텀 키를 의존성에 등록하는 예시 코드에 DependencyValues 확장 부분이 생략되어 있는데, 이를 추가해주면 더욱 완성도 높은 문서가 될 것입니다.
좋은 점
- Firebase Remote Config 연동 시 각 기능에 매핑되어야 하는 키 값들을 표 형태로 명확히 구조화하고, 강제 업데이트와 차단 버전이 동일한 URL 키를 사용한다는 동작 방식을 구체적으로 기술한 점이 훌륭합니다.
(주: 다음 1개 코멘트는 PR diff 범위 밖이라 본문에 모았습니다.)
드롭된 라인 지적
README.md:165— [Suggestion] 커스텀 DependencyKey 구현 예제 코드에서 DependencyValues에 프로퍼티를 등록하는 확장(extension DependencyValues) 코드도 함께 안내되면 사용자가 더 편리하게 연동할 수 있습니다.
리뷰 생성 모델: Gemini 3.5 Flash (High) #3
|
이 PR 은 자동 리뷰를 완료하지 못했습니다 (full 모드 실패 → diff-only 모드 재시도까지 실패).
조치 제안
|
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
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.
변경 사항
forceUpdateAlertDoneLinkURLKey를 함께 사용한다는 점을 명시했습니다.검증
swift test: Swift Testing 12개 테스트 통과./GeneratingDocumentationSite: DocC 정적 사이트 생성 성공git diff --check: 통과