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

fix: changing map language #981

Merged
merged 1 commit into from
Jan 2, 2025
Merged

fix: changing map language #981

merged 1 commit into from
Jan 2, 2025

Conversation

ofirc77
Copy link
Collaborator

@ofirc77 ofirc77 commented Dec 18, 2024

Hi, this is my initial PR for the map translation bug - #950
It currently works only on 'Map by line'. I want to implement that also on 'Time based map' as well.
I think I can improve it mainly with the map URL. I couldn't find better solution at the moment and if we can make the render smoother. I tried to use dependency on the i18n.language but it seems not to work.

Hebrew map:
hebrew_map

English map:
english_map

@ofirc77 ofirc77 requested a review from NoamGaash as a code owner December 18, 2024 21:37
@ofirc77 ofirc77 self-assigned this Dec 18, 2024
@ofirc77 ofirc77 force-pushed the fix_map_translation branch 2 times, most recently from 04cfb9e to 176442d Compare December 18, 2024 21:43
@ofirc77 ofirc77 force-pushed the fix_map_translation branch 2 times, most recently from 8805fc3 to a739da1 Compare December 18, 2024 21:52
@ofirc77 ofirc77 changed the title fix map translation bug fix: map translation bug Dec 18, 2024
@NoamGaash
Copy link
Member

Hi Ofir! Thanks for making this pull request. Would you run npm run lint:fix to sort the lint issue?

@ofirc77 ofirc77 force-pushed the fix_map_translation branch from a739da1 to f95061b Compare December 31, 2024 18:58
@ofirc77 ofirc77 force-pushed the fix_map_translation branch from f95061b to 41ab85d Compare December 31, 2024 19:02
@ofirc77
Copy link
Collaborator Author

ofirc77 commented Dec 31, 2024

Hi @NoamGaash, Done and code was updated :)

@NoamGaash NoamGaash changed the title fix: map translation bug fix: changing map language Jan 2, 2025
Copy link
Member

@NoamGaash NoamGaash left a comment

Choose a reason for hiding this comment

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

Perfect, thanks!
🥳

@ofirc77 ofirc77 merged commit df8a2e3 into main Jan 2, 2025
18 checks passed
@ofirc77 ofirc77 deleted the fix_map_translation branch January 2, 2025 17:46
@NoamGaash
Copy link
Member

@all-contributors please add @ofirc77 as a code contributor 🏅

Copy link
Contributor

@NoamGaash

I've put up a pull request to add @ofirc77! 🎉

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