Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 9, 2025

This PR migrates the map implementation from using default map tiles to OpenStreetMap tiles using the UrlTile component from react-native-maps.

Changes Made

Mobile Implementation (maps.native.tsx):

  • Added UrlTile import from react-native-maps
  • Integrated OpenStreetMap tile layer using https://tile.openstreetmap.org/{z}/{x}/{y}.png
  • Configured appropriate zoom levels (min: 1, max: 19) for optimal performance
  • Preserved all existing functionality:
    • 4 location markers (Tokyo Tower, convenience stores, Tokyo Skytree)
    • Route polyline connecting all locations
    • Original styling and pin colors

Web Fallback (maps.tsx):

  • Updated description text to mention OpenStreetMap instead of generic react-native-maps

Technical Details

The implementation uses the existing react-native-maps library without requiring any additional dependencies. The UrlTile component allows loading custom map tiles from OpenStreetMap's official tile server, providing an open-source alternative to proprietary map services.

<UrlTile
  urlTemplate="https://tile.openstreetmap.org/{z}/{x}/{y}.png"
  maximumZ={19}
  minimumZ={1}
/>

All existing map features remain unchanged, ensuring backwards compatibility while providing the requested OpenStreetMap integration.

Fixes #26.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • api.expo.dev
    • Triggering command: node /home/REDACTED/work/template-mobile-app/template-mobile-app/mobile-app/node_modules/.bin/expo start (dns block)
  • cdp.expo.dev
    • Triggering command: /usr/local/bin/node /home/REDACTED/work/template-mobile-app/template-mobile-app/mobile-app/node_modules/@expo/cli/build/src/utils/telemetry/clients/flushFetchDetached.js /tmp/26822aae2106bc8dc4b9e089bdb3e83c/expo-telemetry.json (dns block)
    • Triggering command: node /home/REDACTED/work/template-mobile-app/template-mobile-app/mobile-app/node_modules/.bin/expo start (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] OpenStreetMapを描画する Migrate map implementation from default tiles to OpenStreetMap using UrlTile Aug 9, 2025
Copilot AI requested a review from naoki-00-ito August 9, 2025 14:16
@naoki-00-ito naoki-00-ito marked this pull request as ready for review August 14, 2025 01:26
@naoki-00-ito naoki-00-ito merged commit dfea533 into develop Aug 14, 2025
2 checks passed
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.

OpenStreetMapを描画する

2 participants