Project

General

Profile

Feature #16361

Translatable pages cleanup

Added by xin 5 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
01/15/2019
Due date:
% Done:

100%

Spent time:
Feature Branch:
xin:feature/16361-translatable-pages-cleanup
Type of work:
Website
Blueprint:
Starter:
Affected tool:

Description

I have some suggestions about translatable pages:

- Stop translating all test news, for the same reasons why alpha, beta and rc test news are already untranslatable (short translation window, test reports should be in English, sometimes not translated).

-Stop translating security/Numerous_security_holes_in_*

-Rename usability_testing_2014_05, report_end_of_2014 and bounties_2013_report

0001-Add-rewrite-rules-for-16361.patch View (1.53 KB) sajolida, 02/20/2019 08:58 PM

History

#1 Updated by xin 5 months ago

  • Assignee set to sajolida

#2 Updated by sajolida 5 months ago

  • Status changed from New to Confirmed
  • Assignee changed from sajolida to xin
  • QA Check set to Info Needed

So, for test we're talking about:

test_3.6~rc1.fr.po
test_asp-beta.fr.po
test_incremental_upgrades.fr.po
test_redesigned_Tails_Greeter.fr.po
test_UEFI.fr.po
test_usb_images.fr.po
test_veracrypt-beta.fr.po

I think that's good. We should remember that, if one day we want to translate some call for testing we'll have to name it something different than test_.

For security announces, I'm fine with not translating security/Numerous_security_holes_in_*.

For report_end_of_2014 I propose to rename this page report_2* so it's already covered by the current rules.

xin: Do you want to try to implement that? Otherwise please reassign it to me for 3.13.

#3 Updated by xin 5 months ago

  • QA Check deleted (Info Needed)

sajolida wrote:

So, for test we're talking about:

[...]

I think that's good. We should remember that, if one day we want to translate some call for testing we'll have to name it something different than test_.

Yes, or create an exception during the short time the page is useful.

Another page to rename: usability_testing_2014_05 to test_usability_2014_05 (a news in French!).

For security announces, I'm fine with not translating security/Numerous_security_holes_in_*.

OK.

For report_end_of_2014 I propose to rename this page report_2* so it's already covered by the current rules.

OK. If we don't change this rules and instead rename pages, we can also rename bounties_2013_report.

xin: Do you want to try to implement that? Otherwise please reassign it to me for 3.13.

Yes, I will work on this after 3.12 release.

#4 Updated by xin 4 months ago

  • Description updated (diff)

#5 Updated by xin 4 months ago

  • Description updated (diff)

#6 Updated by xin 4 months ago

  • Description updated (diff)

#7 Updated by xin 4 months ago

  • Status changed from Confirmed to In Progress
  • Assignee changed from xin to sajolida
  • Target version set to Tails_3.13
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA
  • Feature Branch set to xin:feature/16361-translatable-pages-cleanup

Done, please review.

Things remains to do:
-write redirection rules for the 3 renamed pages
-send updated ikiwiki.setup to sysadmins

#8 Updated by sajolida 4 months ago

It all looks good. I'm reassigning to any sysadmin for them to:

  • Update the ikiwiki.setup on our production website.
  • Apply the patch to puppet-tails.git in attachment.

#9 Updated by intrigeri 4 months ago

  • Update the ikiwiki.setup on our production website.

@sajolida, FYI this is also in puppet-tails.git: https://tails.boum.org/contribute/how/website/production/

#10 Updated by sajolida 4 months ago

@sajolida, FYI this is also in puppet-tails.git: https://tails.boum.org/contribute/how/website/production/

Cool! That's new apparently.

#11 Updated by intrigeri 4 months ago

Cool! That's new apparently.

Yes, that's the case only since we self-host our website (last November).

#12 Updated by groente 4 months ago

  • Assignee changed from groente to sajolida
  • QA Check changed from Ready for QA to Info Needed

sajolida wrote:

It all looks good. I'm reassigning to any sysadmin for them to:

  • Update the ikiwiki.setup on our production website.
  • Apply the patch to puppet-tails.git in attachment.

Update the ikiwiki.setup how/to what exactly? I'm missing the updated version mentioned in '-send updated ikiwiki.setup to sysadmins' here...

#13 Updated by sajolida 4 months ago

  • Assignee changed from sajolida to groente
  • QA Check deleted (Info Needed)

To the version in the branch referenced in the metadata of this ticket: xin:feature/16361-translatable-pages-cleanup.

#14 Updated by intrigeri 4 months ago

To the version in the branch referenced in the metadata of this ticket: xin:feature/16361-translatable-pages-cleanup.

… and "xin:" means https://git.tails.boum.org/xin/tails

In puppet-tails.git, ikiwiki.setup is a template, so if tech writers don't want follow https://tails.boum.org/contribute/how/website/production/, then groente will need to extract the changes from master to xin's branch, and apply them to the template.

#15 Updated by groente 4 months ago

  • Status changed from In Progress to Resolved
  • % Done changed from 50 to 100

merged & deployed

#16 Updated by xin 4 months ago

  • Status changed from Resolved to In Progress
  • Assignee changed from groente to sajolida
  • % Done changed from 100 to 80

Now 2 things remains to do:
- merge my branch into master (more urgent since updated ikiwiki.setup was deployed)
- apply 0001-Add-rewrite-rules-for-16361.patch to puppet-tails.git

#17 Updated by sajolida 4 months ago

  • Assignee changed from sajolida to bertagaz

I did the merge in master. I'll let sysadmins apply 0001-Add-rewrite-rules-for-16361.patch to puppet-tails.git.

@bertagaz: Are you still on duty?

#18 Updated by bertagaz 4 months ago

  • Assignee changed from bertagaz to sajolida

sajolida wrote:

I did the merge in master. I'll let sysadmins apply 0001-Add-rewrite-rules-for-16361.patch to puppet-tails.git.

bertagaz: Are you still on duty?

Yes. Patch is deployed and working. Assigning back to you.

#19 Updated by xin 4 months ago

  • % Done changed from 80 to 90

It works but .po files was generated again by commit 457625cd9974a5b52093fe1a9a980b16c10472bf, can you revert it? Thanks!
If think it should fix the current problem with lists like in https://tails.boum.org/security/index.en.html

#20 Updated by xin 4 months ago

I forgot the 3.12.1 release happened after my work, so Numerous_security_holes_in_3.12.*.po files have to be removed too. Can you do it or I update my branch for that?
After that all is done.

#21 Updated by sajolida 4 months ago

  • Status changed from In Progress to Resolved
  • Assignee deleted (sajolida)

Works, thanks!

#22 Updated by sajolida 4 months ago

Done in b28e9bf667.

#23 Updated by xin 4 months ago

  • % Done changed from 90 to 100

Also available in: Atom PDF