Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.0.0 - localize #127

Merged
merged 27 commits into from
Jul 15, 2022
Merged

1.0.0 - localize #127

merged 27 commits into from
Jul 15, 2022

Conversation

mapsam
Copy link
Contributor

@mapsam mapsam commented Jul 15, 2022

This creates a new function named localize, intended for finding language and worldviews within an encoded vector tile and modifying properties to "translate" them, as well as drop unused translations to keep tile size small.

The original composite function is now accessible via:

const { composite, localize } = require('@mapbox/vtcomposite');

gerdie and others added 26 commits January 19, 2022 11:06
Co-authored-by: John Klancer <john.klancer@mapbox.com>
Co-authored-by: John Klancer <john.klancer@mapbox.com>
* 1st pass at adding worldview into params

* - basic worldview support

* [publish binary] 1.0.0-dev2

* - adds support for comma-separated worldviews

* [publish binary] 1.0.0-dev3

* testing changes:
- combine lang + worldview
- verify ID is not added
- check geometry

* linting internationalize test

* moving test util function

* readme tweaks;

* removing todo comments

* very wip to replicate legacy worldview features

* wip2]

* cleanup cleanup everybody everywhere

* i18n worldview tests

* uncomment binding

* tidy and format

* v1.0.0-dev4 [publish binary]

* uncomment binding [publish binary]

* test with whacky/mismatching worldview values

* update and split docs

* update xcode12.4 and remove CXX flags

* revert xcode changes

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* utils functions for split and intersect

* uncomment binding.gyp

* std::begin -> begin() / end(), var renaming

* swap order of build_internationalized_feature parameters

* tidy and format

* revert clang versions

* 1.0.0-dev5 [publish binary]

Co-authored-by: mapsam <matthews.sam@gmail.com>
Co-authored-by: Sam Matthews <sam@mapbox.com>
* internationalize -> localize

* uncomment binding

* Update src/vtcomposite.cpp

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update src/vtcomposite.cpp

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* 1.0.0-dev6 [publish binary]

* Update README.md

Co-authored-by: Rajesh Amradi <rajesh.amradi@mapbox.com>

* use mvt-fixtures 3.8.0

Co-authored-by: John Klancer <john.klancer@mapbox.com>
Co-authored-by: Rajesh Amradi <rajesh.amradi@mapbox.com>
* localize(params), worldview_property option

* worldview_defaults, tests with inline mvt fixtures

* use mvt-fixtures 3.9.0

* update README

* remove worldview_defaults param, update readme + tests

* update param validation tests

* review feedback

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* Update README.md

Co-authored-by: John Klancer <john.klancer@mapbox.com>

* tidy and format

* 1.0.0-dev7 [publish binary]

Co-authored-by: John Klancer <john.klancer@mapbox.com>
* language_property, language_prefix

* add params.language_property and params.language_prefix test

* add language params docs

* make format

* fix test name
@mapsam mapsam requested a review from a team July 15, 2022 15:21
Copy link

@davidweimapbox davidweimapbox left a comment

Choose a reason for hiding this comment

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

wow, didn't realize it's a such big change PR. Great work! I am assuming most of the changes are taken from the already approved PRs. LGTM. thanks!

@mapsam mapsam merged commit 0ea5136 into master Jul 15, 2022
@mapsam mapsam deleted the i18n branch July 15, 2022 19:05
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.

3 participants