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

Map may not be localising Vocab directory values correctly? #218

Closed
wu-lee opened this issue Aug 16, 2023 · 3 comments
Closed

Map may not be localising Vocab directory values correctly? #218

wu-lee opened this issue Aug 16, 2023 · 3 comments
Assignees

Comments

@wu-lee
Copy link
Contributor

wu-lee commented Aug 16, 2023

Describe the bug
The DotCoop 2023 demo map uses the country ID as a directory field. When switching to a supported language for the UI, say ES, the UI shows Spanish but the country names seem to remain English.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://dev.maps.solidarityeconomy.coop/temp/dotcoop2023/?language=FR
  2. Open the directory sidebar
  3. Check the country names.

Expected behavior
The directory should be labelled "Annuaire" and it should have the first three values "Argentine", "Australie" and "Austriche".

What's seen
Although the label is correct the first three values are "Argentina", "Australia" and "Austria". Similarly for Spanish (language=ES), the names are in English.

Desktop:

  • OS: Linux
  • Browser FF
  • Version 116.0.2
@wu-lee wu-lee self-assigned this Aug 16, 2023
@ColmDC
Copy link
Contributor

ColmDC commented Sep 8, 2023

I cannot replicate this in Firefox on Windows on a MM v3.0.9. @wu-lee

@wu-lee
Copy link
Contributor Author

wu-lee commented Sep 11, 2023

I've actually found a bug related this when working on #200 - the code has a function to look up vocab terms with a parameter for the language which defaults to the fallback language (English). The parameter was omitted and so was always the fallback language.

The reason why you couldn't replicate it is that the bug was introduced after v3.0.9, in fact in the branch for #200 (appearing in v3.1.0). It appeared in, and was fixed in updates to this change: e45b88c

I've also made the parameter non-optional, just to avoid that in the future. Here e45b88c

@ColmDC
Copy link
Contributor

ColmDC commented Sep 11, 2023

Can't replicate on 3.1.0, so closing.

@ColmDC ColmDC closed this as completed Sep 11, 2023
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

No branches or pull requests

2 participants