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

(fixes #5855) fetch item label in nearby based on user configured lan… #5858

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

ChristoJobyAntony
Copy link
Contributor

Fix the Nearby functionality to fetch Item tag based on user's preferred language

Currently, the nearby feature, fetches all the labels to be in English. However, the expected behaviour would be to fetch based on user's preferred language.

Fixes #5855

What changes did you make and why?

The only change made was to the query file. It was updated to fetch labels based on {LANG} variable.

Tests performed (required)

Tested ProdDebug on Pixel 7 with API level 34 to ensure that Wiki Items were presented with labels in user language if available. Tested for the Wiki Item Bust of Gustave Eiffel. The languages tested were, French, Armenian, Arabic & Turkish. Before each screenshot was taken, language preference was updated and then the item was located in Nearby.

@ChristoJobyAntony
Copy link
Contributor Author

@nicolas-raoul would you be able to review this PR ?

Copy link
Member

@nicolas-raoul nicolas-raoul left a comment

Choose a reason for hiding this comment

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

Thanks! :-)

@nicolas-raoul nicolas-raoul merged commit 3ac608c into commons-app:main Oct 16, 2024
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.

SPARQL: Item label does not seem to follow configured language, "en" hard-coded
2 participants