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

Language specific label formats #1617

Merged
merged 2 commits into from
Mar 31, 2022
Merged

Conversation

orangejulius
Copy link
Member

This PR includes the required API changes to use our new language specific label formats from pelias/labels#52 and pelias/labels#51.

The code changes required to enable this is just one line, but for now this PR also includes a change to point the pelias-labels package to a branch. We shouldn't merge this PR until pelias/labels#52 is merged and released.

We also might want to add a quick test or two that the language values make it into the label generator.

@orangejulius orangejulius force-pushed the language-specific-label-formats branch 2 times, most recently from 3e3d420 to 539a054 Compare March 22, 2022 16:29
This is required for language-specific label functionality
This allows labels to be customized depending on the display language
selected.
@orangejulius orangejulius force-pushed the language-specific-label-formats branch from 539a054 to b5e0a62 Compare March 22, 2022 16:59
@orangejulius orangejulius marked this pull request as ready for review March 22, 2022 18:19
@orangejulius orangejulius changed the title WIP: Language specific label formats Language specific label formats Mar 31, 2022
@orangejulius orangejulius merged commit 8d72ac1 into master Mar 31, 2022
@orangejulius orangejulius deleted the language-specific-label-formats branch March 31, 2022 16:16
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.

1 participant