Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

QMAPS-2113 use qwant's "l" laguage cookie #1082

Merged
merged 9 commits into from
May 11, 2021

Conversation

xem
Copy link
Contributor

@xem xem commented May 5, 2021

Description

  • Use Qwant's "l=..." cookie (if possible) as the interface language of Qwant Maps
  • Fallback on the first available/useable language in the accepted-languages browser header (same as before)
  • Fallback on Qwant Maps default language if nothing fits (same as before)

Why

Better coherence with Qwant.com settings

bin/middlewares/user_language.js Show resolved Hide resolved
bin/middlewares/user_language.js Outdated Show resolved Hide resolved
@xem xem requested a review from amatissart May 5, 2021 16:45
bin/app.js Outdated Show resolved Hide resolved
@xem xem changed the title QMAPS-2113 use qwant's "l" laguage QMAPS-2113 use qwant's "l" laguage cookie May 10, 2021
@bbecquet
Copy link
Contributor

@xem
From your last commits, it looks like your prettier config doesn't work anymore :/
It should be entirely automatic, saving you from the pain of doing that by hand. We'll find how to restore it.

@xem xem requested a review from bbecquet May 11, 2021 09:11
@xem xem merged commit 57f235e into master May 11, 2021
@amatissart amatissart deleted the QMAPS-2113-use-qwant-interface-language-cookie branch June 29, 2021 15:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants