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

ui: multilang offroad alerts #28512

Merged
merged 16 commits into from Jun 22, 2023

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Jun 12, 2023

generate a translations include file selfdrive/ui/translations/generated_translations.h to multilingualize the offroad alerts

@deanlee deanlee marked this pull request as draft June 12, 2023 18:47
@deanlee deanlee force-pushed the ui_multilang_offroad_alert branch from 60551a8 to e599c98 Compare June 12, 2023 18:49
@deanlee deanlee marked this pull request as ready for review June 12, 2023 18:50
@deanlee deanlee force-pushed the ui_multilang_offroad_alert branch 3 times, most recently from c6aac45 to f1e9fff Compare June 12, 2023 20:36
@deanlee
Copy link
Contributor Author

deanlee commented Jun 12, 2023

@sshane this is ready for review

selfdrive/ui/.gitignore Outdated Show resolved Hide resolved
selfdrive/ui/update_translations.py Show resolved Hide resolved
selfdrive/ui/update_translations.py Outdated Show resolved Hide resolved
selfdrive/ui/update_translations.py Outdated Show resolved Hide resolved
selfdrive/ui/update_translations.py Outdated Show resolved Hide resolved
@deanlee deanlee requested a review from sshane June 13, 2023 07:54
selfdrive/controls/lib/alerts_offroad.json Outdated Show resolved Hide resolved
selfdrive/controls/lib/alerts_offroad.json Outdated Show resolved Hide resolved
selfdrive/ui/.gitignore Outdated Show resolved Hide resolved
selfdrive/ui/update_translations.py Show resolved Hide resolved
selfdrive/ui/update_translations.py Outdated Show resolved Hide resolved
@sshane
Copy link
Contributor

sshane commented Jun 13, 2023

LGTM besides those things! @adeebshihadeh what do you think?

@sshane
Copy link
Contributor

sshane commented Jun 15, 2023

can you rebase one more time @deanlee?

@deanlee
Copy link
Contributor Author

deanlee commented Jun 15, 2023

done

@deanlee deanlee force-pushed the ui_multilang_offroad_alert branch 3 times, most recently from acdd144 to 14c5f66 Compare June 18, 2023 05:19
@deanlee deanlee force-pushed the ui_multilang_offroad_alert branch from 14c5f66 to b06b4bb Compare June 21, 2023 05:51
@incognitojam
Copy link
Contributor

Tested, seems to work!

@incognitojam incognitojam merged commit 2b800cf into commaai:master Jun 22, 2023
19 of 20 checks passed
@deanlee deanlee deleted the ui_multilang_offroad_alert branch June 22, 2023 14:27
@sshane sshane mentioned this pull request Aug 7, 2023
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants