[Feat/#77] 파이어베이스 연동 추가#78
Conversation
Summary of ChangesHello @imtaejugkim, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Firebase 서비스를 프로젝트에 통합하고, 애플리케이션 ID를 프로젝트의 실제 이름에 맞춰 변경하며, 디자인 시스템 내의 네이밍 불일치를 수정하여 전반적인 코드 베이스의 일관성과 기능을 향상시키는 것을 목표로 합니다. 이를 통해 향후 Firebase 관련 기능 개발을 위한 기반을 마련하고, 프로젝트의 명확성을 높였습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
…to feat/#77-firebase-connect
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이 PR은 Firebase 연동을 추가하고 관련 네이밍을 변경하는 작업을 포함하고 있습니다. Firebase 설정이 잘 추가되었지만, 몇 가지 중요한 점을 발견했습니다. 가장 중요한 것은 google-services.json 파일에 API 키와 같은 민감한 정보가 포함되어 있어 이를 버전 관리에 포함하는 것은 심각한 보안 위험을 초래할 수 있다는 점입니다. 또한, 테마 파일에서 오타로 인해 잘못된 오류 메시지가 표시될 수 있는 버그를 발견했습니다. 아래의 상세 리뷰 코멘트를 확인하고 수정해 주시기 바랍니다.
| { | ||
| "project_info": { | ||
| "project_number": "842640708296", | ||
| "project_id": "togedy-e4645", | ||
| "storage_bucket": "togedy-e4645.firebasestorage.app" | ||
| }, | ||
| "client": [ | ||
| { | ||
| "client_info": { | ||
| "mobilesdk_app_id": "1:842640708296:android:acbde575012114d11e0bfd", | ||
| "android_client_info": { | ||
| "package_name": "com.together.study" | ||
| } | ||
| }, | ||
| "oauth_client": [], | ||
| "api_key": [ | ||
| { | ||
| "current_key": "AIzaSyAXskBWB0XpHXWCBOMJfIWAW76iNSZqz9s" | ||
| } | ||
| ], | ||
| "services": { | ||
| "appinvite_service": { | ||
| "other_platform_oauth_client": [] | ||
| } | ||
| } | ||
| } | ||
| ], | ||
| "configuration_version": "1" | ||
| } No newline at end of file |
There was a problem hiding this comment.
보안 위험: google-services.json 파일에는 API 키와 같은 민감한 정보가 포함되어 있습니다. 이 파일을 Git 저장소에 직접 커밋하는 것은 매우 위험합니다. API 키가 노출되면 악의적인 사용자가 서비스를 남용할 수 있습니다. 이 파일은 .gitignore에 추가하여 버전 관리에서 제외하고, CI/CD 환경 변수나 다른 보안 메커니즘을 통해 안전하게 관리해야 합니다. 이 파일을 저장소에서 제거하고 노출된 키를 무효화하는 것을 강력히 권장합니다.
|
|
||
| private val LocalTogedyTypography = staticCompositionLocalOf<TogedyTypography> { | ||
| error("No NapzakTypography provided") | ||
| error("No TogedyColors provided") |
There was a problem hiding this comment.
오타 수정: LocalTogedyTypography의 기본값을 설정하는 부분에서 오류 메시지가 No TogedyColors provided로 잘못 작성되었습니다. 이는 디버깅 시 혼란을 줄 수 있으므로 No TogedyTypography provided로 수정해야 합니다.
| error("No TogedyColors provided") | |
| error("No TogedyTypography provided") |
1 participant
📝 Summary
파이어베이스 연동 추가
네이밍 변경
🔍 Related Issues
📸 Screenshots (선택)
✅ Checklist (남아있는 사항)
🗒 Additional Notes
배포 테스트 진행하겠습니다