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

feat(app): implement system language modal #16507

Merged
merged 6 commits into from
Oct 17, 2024
Merged

Conversation

brenthagen
Copy link
Contributor

Overview

implements the system language modal that displays on first boot of the desktop app or when the language of the computer running the desktop app changes.

closes PLAT-536

Screen Shot 2024-10-17 at 10 56 06 AM Screen Shot 2024-10-17 at 10 57 07 AM

Test Plan and Hands on Testing

unit tests for the modal and system language matching to supported locales.

Changelog

  • Implements system language modal

Review requests

with localization feature flag on, observe modal on first boot prior to setting app language. config.json can be edited directly in Library/Application Support/Opentrons/config.json to check different scenarios.

Risk assessment

low

@brenthagen brenthagen requested a review from a team October 17, 2024 15:05
@brenthagen brenthagen requested a review from a team as a code owner October 17, 2024 15:05
@brenthagen brenthagen requested review from ncdiehl11 and removed request for a team and ncdiehl11 October 17, 2024 15:05
Copy link
Contributor

@vegano1 vegano1 left a comment

Choose a reason for hiding this comment

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

This looks great!

@brenthagen brenthagen merged commit bfc8a37 into edge Oct 17, 2024
36 of 37 checks passed
@brenthagen brenthagen deleted the app_system-language-modal branch October 17, 2024 21:49
TamarZanzouri pushed a commit that referenced this pull request Oct 18, 2024
implements the system language modal that displays on first boot of the
desktop app or when the language of the computer running the desktop app
changes.

closes PLAT-536
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