mapbox / mapbox/MapboxGeocoder.swift

Read API base URL from Info.plist

Offen
#107 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Swift
Sterne
133
Forks
46
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie in MapboxGeocoder/MBGeocoder.swift ungefähr bei den Zeilen 147–154 und vergleichen Sie dort die bestehende Suche nach dem Access-Token in Info.plist mit der Fallback-Host-Logik. Die Änderung ist abgeschlossen, wenn ein weggelassener Host api.mapbox.com oder api.mapbox.cn aus MGLMapboxAPIBaseURL auswählt, während ein ausdrücklich angegebener Host weiterhin eine Überschreibung bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
swift
Bereich
mobile-dev
Issue-Typ
Feature
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.