Project

General

Profile

Feature #29669

Umbrella #29625: Add translation markers to the UI

Add translation markers to the storage section

Added by Erin Clark over 2 years ago. Updated over 2 years ago.

Status:
Done
Priority:
Important
Assignee:
Peter Southwell
Category:
GUI (new)
Estimated time:
Severity:
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

The storage section needs to be translatable. Any text that goes into the tooltip component will automatically be translated, all you need to do is add the T() marker, see Account Users for an example. In order to extract strings that are not in tooltips (either in the html or the ts files) you will need to pass them through the ngx-translate service, see examples from the login page or the admin bar. After they are passed through you should be able to run 'npm run extract' and the new keys will appear in the PO files. The volume manager will be translated in its own ticket


Related issues

Copied from FreeNAS - Feature #29665: Add translation markers to the sharing sectionDone2018-03-12
Copied to FreeNAS - Feature #29673: Add translation markers to the volume managerDone2018-03-12

History

#1 Updated by Erin Clark over 2 years ago

  • Copied from Feature #29665: Add translation markers to the sharing section added

#2 Updated by Erin Clark over 2 years ago

  • Copied to Feature #29673: Add translation markers to the volume manager added

#3 Updated by Peter Southwell over 2 years ago

  • Status changed from Not Started to In Progress

#4 Updated by Peter Southwell over 2 years ago

  • Status changed from In Progress to Done

#5 Updated by Dru Lavigne over 2 years ago

  • Status changed from Done to In Progress
  • Target version changed from 11.2-RC2 to Master - FreeNAS Nightlies

#6 Updated by Peter Southwell over 2 years ago

  • Status changed from In Progress to Done
  • % Done changed from 0 to 100

Howdie Dru. Apologies if Im wrognly changing this back.

The PR was broken due to a non code related issue (NOt enough hard drive space to do checkout errors)... So I did write and finish the code for this.

Is it because the PR didn't clear yet? No worries.. Happy to change it back if Im wrongly re-changing it.

#7 Updated by Dru Lavigne over 2 years ago

  • Needs Merging changed from Yes to No

Also available in: Atom PDF