Skip to content

Conversation

@DandelionQZ
Copy link
Collaborator

⚙️ Related ISSUE Number

#74



📄 Work Description

google play console에서 안드로이드 내부 테스트를 위한 설정을 했습니다.

  • 환경 변수 관리를 위한 dotenv 추가
    앱 전체에서 환경 변수를 로드할 수 있도록 app.config.js에서 이를 호출하도록 설정
  • 프로젝트 버전 수정
    새로운 개발 단계에 맞춰 일관성을 유지하기 위해 package.json, package-lock.json 및 app.config.js에서 프로젝트 버전을 1.0.0에서 0.1.0으로 업데이트
  • 호환성을 위한 react-native-pager-view 의존성 다운그레이드
    호환성 또는 안정성 문제를 해결하기 위해 package.json 및 package-lock.json 모두에서 react-native-pager-view 버전을 7.0.2에서 6.9.1로 다운그레이드
  • 안드로이드 빌드 설정 업데이트
    안드로이드 프로덕션 빌드 시 app-bundle 빌드 유형을 사용하도록 eas.json을 업데이트



📷 Screenshot



💬 To Reviewers



🔗 Reference

@DandelionQZ DandelionQZ self-assigned this Dec 31, 2025
Copilot AI review requested due to automatic review settings December 31, 2025 17:26
@DandelionQZ DandelionQZ linked an issue Dec 31, 2025 that may be closed by this pull request
3 tasks
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR configures the TinyBite app for Android internal testing on Google Play Console. The main changes include adding environment variable management support, adjusting project versioning, and updating build configurations.

Key Changes:

  • Added dotenv package for environment variable management and integrated it into app.config.js
  • Downgraded project version from 1.0.0 to 0.1.0 to align with the current development stage
  • Downgraded react-native-pager-view from 7.0.2 to 6.9.1 for compatibility/stability

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
TinyBite/package.json Added dotenv dependency, downgraded react-native-pager-view version, and updated project version to 0.1.0
TinyBite/package-lock.json Updated lockfile to reflect package.json changes with new dependency versions
TinyBite/eas.json Configured Android production builds to use app-bundle build type for Play Store deployment
TinyBite/app.config.js Added dotenv configuration at the top and updated project version to 0.1.0
TinyBite/.gitignore Modified iOS and Android directory ignore patterns from leading slash to trailing slash format
TinyBite/.vscode/settings.json Added Java-related VS Code configuration for null analysis and build configuration updates
TinyBite/assets/images/icon.png Updated application icon (binary file)
Files not reviewed (1)
  • TinyBite/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +1 to +2
require("dotenv").config();

Copy link

Copilot AI Dec 31, 2025

Choose a reason for hiding this comment

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

The dotenv.config() call is unnecessary in this context. Expo automatically loads environment variables prefixed with EXPO_PUBLIC_ from .env files. The existing code only uses process.env.EXPO_PUBLIC_* variables (lines 43, 63, 64), which Expo handles natively without requiring the dotenv package. Consider removing the dotenv dependency and this config call unless you plan to use non-EXPO_PUBLIC_ prefixed environment variables.

Suggested change
require("dotenv").config();

Copilot uses AI. Check for mistakes.
Comment on lines +44 to +45
ios/
android/
Copy link

Copilot AI Dec 31, 2025

Choose a reason for hiding this comment

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

The gitignore pattern change from /ios and /android to ios/ and android/ alters the matching behavior. The leading slash /ios matches only the ios directory at the repository root, while ios/ (with trailing slash) matches any directory named ios at any level in the repository. If you only want to ignore the root-level ios and android directories (which is typical for React Native/Expo projects), the original /ios and /android patterns were correct. Consider reverting to the original patterns with leading slashes.

Suggested change
ios/
android/
/ios
/android

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 앱 배포를 위한 설정

2 participants