Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[ios, macos] Change "map ID" to "tileset ID" #15116

Merged
merged 4 commits into from
Jul 17, 2019
Merged

Conversation

HeyStenson
Copy link
Contributor

Background: We're changing instances of map ID to be the more accurate term tileset ID across all Mapbox documentation. @1ec5 also found a number of instances where the docs in this repo refer to a style's map ID, which is more accurately a style URL.

Per #14551, this PR:

  • Updates map ID to tileset ID where necessary
  • Updates style's map ID to style URL where necessary

@captainbarbosa or @1ec5 to review, please! 🙏

@HeyStenson HeyStenson requested review from 1ec5, captainbarbosa and a team July 12, 2019 22:59
Copy link
Contributor

@captainbarbosa captainbarbosa left a comment

Choose a reason for hiding this comment

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

While this doesn't affect any APIs, I'm wondering if we should include a changelog entry (in platform/{ios/macos}/CHANGELOG.md noting this change in language. I could go either way on that decision though, so this looks great regardless! 👍

point to a JSON file that conforms to the
<a href="https://github.com/mapbox/tilejson-spec/">TileJSON specification</a>.

If a Mapbox URL is specified, this source uses a tile size of 256. For all
other tile sets, the default value is 512. (See the
other tilesets, the default value is 512. (See the
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks :)

@HeyStenson
Copy link
Contributor Author

@captainbarbosa Great idea! Changelog entries added.

Also, ummm... I can't figure out why the CircleCI tests are failing 🙈 Any ideas?

@friedbunny friedbunny added documentation iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS labels Jul 15, 2019
@friedbunny friedbunny added this to the release-picklejuice milestone Jul 15, 2019
@friedbunny
Copy link
Contributor

CI failures look like unrelated flakiness — I’ll rerun those jobs.

@friedbunny friedbunny changed the title [DNM] Change "map ID" to "tileset ID" [ios, macos] Change "map ID" to "tileset ID" Jul 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants