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

Update tile URL for HERE traffic #691

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

omartijn
Copy link

@omartijn omartijn commented Jan 4, 2025

This fixes 400 errors when trying to use HERE traffic tiles with a newer account. The old URL was deprecated and only works in older accounts.

Note: I do not have access to an old account and I cannot be sure that this works correctly for old accounts. Please test this before merging.

This fixes 400 errors when trying to use HERE traffic tiles with a newer
account. The old URL was deprecated and only works in older accounts.
@rinigus
Copy link
Owner

rinigus commented Jan 5, 2025

Thank you very much! I will review it in the next few days. Note that tools subfolder has some scripts that may help with HERE languages maps generation.

@rinigus
Copy link
Owner

rinigus commented Jan 20, 2025

it looks to be mostly working except if you use mar default language: local. In this case, lang=mul is added to URL and I am getting errors (400) with a message

  "details": [
    {
      "message": "The 'lang' parameter in 'query' has an error: string doesn't match the regular expression \"^[a-z]{2}(-[a-zA-Z]{1,4})?$\""
    }

I wonder what is the analog of mul in the new API? This is using old account

@rinigus
Copy link
Owner

rinigus commented Jan 20, 2025

PS: sorry for delay!

@omartijn
Copy link
Author

Well, according to the Migration Guide they now use BCP 47, it mentions the following:

The 'mul' (Multiple) primary language subtag identifies content in multiple languages. This subtag SHOULD NOT be used when a list of languages or individual tags for each content element can be used instead. For example, the 'Content- Language' header [RFC3282] allows a list of languages to be used, not just a single language tag.

I cannot find another suggestion as to what to use. I guess you want to use this to get the native names (i.e. the ones local to what you are looking at). I think Pure Maps should get the language code for the country being viewed here.

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.

2 participants