Skip to content

Phoebus is not fully translated - #3982

Merged
georgweiss merged 8 commits into
ControlSystemStudio:masterfrom
lcaouen:update_french_translation
Oct 24, 2026
Merged

georgweiss merged 8 commits into
ControlSystemStudio:masterfrom
lcaouen:update_french_translation

Conversation

@lcaouen

@lcaouen lcaouen commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Checklist

change java files to add missing Messages.java class for the translation

  • Testing:

    • Check visually the GUI was translated into French
  • Documentation:

    • No change in the documentation

@georgweiss

Copy link
Copy Markdown
Collaborator

@lcaouen, can you please check the Sonarqube report, there is a duplicate import in a Messages class.

@sonarqubecloud

Copy link
Copy Markdown

@lcaouen
lcaouen marked this pull request as ready for review October 23, 2026 21:59
@lcaouen

lcaouen commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

@georgweiss I have corrected the duplicated import and also some missing code.
I think it's ready to merge

@georgweiss
georgweiss merged commit a67022c into ControlSystemStudio:master Oct 24, 2026
9 checks passed
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