mapbox / mapbox/MapboxGeocoder.swift

Read API base URL from Info.plist

Đang mở
#107 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

enhancement
Ngôn ngữ chính
Swift
Star
133
Fork
46
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

If no host is passed into Geocoder(accessToken:host:), the fallback hostname should be either api.mapbox.com or api.mapbox.cn depending on the value of MGLMapboxAPIBaseURL in Info.plist:

https://github.com/mapbox/MapboxGeocoder.swift/blob/faa288dcf09f9dbcab1a54e61e741932f3945239/MapboxGeocoder/MBGeocoder.swift#L154

This would be consistent with how we read the access token out of Info.plist:

https://github.com/mapbox/MapboxGeocoder.swift/blob/faa288dcf09f9dbcab1a54e61e741932f3945239/MapboxGeocoder/MBGeocoder.swift#L147

This will make it easier for developers to keep their map and geocoder usage in sync, since Geocoder.shared would automatically pick up the API endpoint specified in Info.plist. Developers who need a different value for the geocoder can still override the API base URL when creating a separate Geocoder instance.

/cc @pdgoodman

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu trong MapboxGeocoder/MBGeocoder.swift, khoảng các dòng 147-154, bằng cách so sánh việc tra cứu access-token hiện có trong Info.plist với logic host dự phòng. Thay đổi được hoàn tất khi host bị bỏ qua chọn api.mapbox.com hoặc api.mapbox.cn từ MGLMapboxAPIBaseURL, trong khi host được cung cấp rõ ràng vẫn là một giá trị ghi đè.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
swift
Lĩnh vực
mobile-dev
Loại issue
Tính năng
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.