Skip to content

baseURLを本番APIサーバーURLに設定#62

Merged
miyakoshi-3854 merged 2 commits intomainfrom
hotfix/deploy-error
Jul 26, 2025
Merged

baseURLを本番APIサーバーURLに設定#62
miyakoshi-3854 merged 2 commits intomainfrom
hotfix/deploy-error

Conversation

@nkoji21
Copy link
Copy Markdown
Member

@nkoji21 nkoji21 commented Jul 26, 2025

📝 変更内容

何を変更したか

なぜ変更したか


🧪 テスト・確認項目

動作確認

  • プルリクエストにラベルを追加したか
  • アサインに自分を追加したか
  • ローカル環境で動作確認済み
  • 既存機能に影響がないことを確認
  • モバイル表示を確認(該当する場合)

📸 スクリーンショット(UI変更がある場合)

Before After
Before After

💡 補足事項


🔗 関連Issue

Closes #

@nkoji21 nkoji21 self-assigned this Jul 26, 2025
@nkoji21 nkoji21 added the 🐛 バグ 動かない、おかしいなど、不具合の修正タスク label Jul 26, 2025
Copilot AI review requested due to automatic review settings July 26, 2025 22:27
@nkoji21 nkoji21 requested a review from miyakoshi-3854 as a code owner July 26, 2025 22:27
@miyakoshi-3854 miyakoshi-3854 merged commit 34608fc into main Jul 26, 2025
1 check passed
@miyakoshi-3854 miyakoshi-3854 deleted the hotfix/deploy-error branch July 26, 2025 22:28
Copy link
Copy Markdown

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 centralizes API configuration by creating a dedicated configuration module and replacing direct environment variable usage with a consistent API configuration object. The change sets the base URL to the production API server.

  • Creates a centralized API configuration module (api.ts)
  • Replaces import.meta.env.VITE_API_URL usage with API_CONFIG.baseUrl in components
  • Sets production API server URL as the default base URL

Reviewed Changes

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

File Description
apps/web/src/lib/api.ts Creates new API configuration module with production base URL
apps/web/src/components/photo/PhotoPreview.tsx Updates fetch call to use centralized API config
apps/web/src/components/mode/ModeScreen.tsx Updates fetch call to use centralized API config

Comment thread apps/web/src/lib/api.ts
Comment thread apps/web/src/lib/api.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 バグ 動かない、おかしいなど、不具合の修正タスク

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants