Skip to content

Update embassy.json#842

Closed
kjonosm wants to merge 1 commit intoopenstreetmap:mainfrom
kjonosm:patch-34
Closed

Update embassy.json#842
kjonosm wants to merge 1 commit intoopenstreetmap:mainfrom
kjonosm:patch-34

Conversation

@kjonosm
Copy link
Copy Markdown
Contributor

@kjonosm kjonosm commented Mar 26, 2023

Order values alphabetically

Order values alphabetically
@github-actions
Copy link
Copy Markdown

🍱 Preview the tagging presets of this pull request here: https://pr-842--ideditor-presets-preview.netlify.app/id/dist/#locale=en.

@tyrasd
Copy link
Copy Markdown
Member

tyrasd commented Mar 27, 2023

The order was actually chosen on purpose in #343 to reflect the relative frequencies of the different values (see taginfo). As the alphabetic ordering would be broken by the translation strings (already in English), it is of little benefit for users. I'd prefer to keep it as is.

@tyrasd tyrasd closed this Mar 27, 2023
@tyrasd tyrasd added the wontfix This will not be worked on label Mar 27, 2023
@kjonosm kjonosm deleted the patch-34 branch March 27, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants