Skip to content

[Security Overview FR version] Add the FR version for each documents in the Security Overview section#3681

Merged
SC-Jerome-S merged 6 commits into
masterfrom
feat/security-overview-FR-version-creation
Jun 30, 2026
Merged

[Security Overview FR version] Add the FR version for each documents in the Security Overview section#3681
SC-Jerome-S merged 6 commits into
masterfrom
feat/security-overview-FR-version-creation

Conversation

@SC-Jerome-S

@SC-Jerome-S SC-Jerome-S commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Add French translations to Security Overview documentation
This PR adds complete French translations to all Security Overview documentation files, making them bilingual (English/French).

Changes
Files modified: 15 security overview documents in overview

2000-01-01-application-security.md
2000-01-01-auditing.md
2000-01-01-backups.md
2000-01-01-business-disaster.md
2000-01-01-code-security.md
2000-01-01-compliance.md
2000-01-01-connectivity.md
2000-01-01-data-protection.md
2000-01-01-employee-security.md
2000-01-01-incident-support.md
2000-01-01-monitoring.md
2000-01-01-network-security.md
2000-01-01-operational-security.md
2000-01-01-physical-security.md
2000-01-01-third-parties.md

Implementation
Each file now includes:

Bilingual navigation links at the top (English version / Version française)
Anchor tags and for direct linking to each language version
Complete French translations of all content, maintaining technical accuracy
Updated modification dates to 2026-04-16
This ensures French-speaking users can access security documentation in their native language while preserving the original English content.

@benjaminach
benjaminach requested a review from Frzk April 17, 2026 08:59
@SC-Jerome-S

Copy link
Copy Markdown
Contributor Author

@yanjost can you take a look?

@semgrep-code-scalingo

Copy link
Copy Markdown

Legal Risk

The following dependencies were released under a license that
has been flagged by your organization for consideration.

Recommendation

While merging is not directly blocked, it's best to pause and consider what it means to use this license before continuing. If you are unsure, reach out to your security team or Semgrep admin to address this issue.

MPL-2.0

@SC-Jerome-S

SC-Jerome-S commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

Side note: I was going to propose a very small change to the jekyll-extlinks.rb but I think this is not the place. (my build was failling). So I removed the change to focus only on the security documents

@SC-Jerome-S
SC-Jerome-S force-pushed the feat/security-overview-FR-version-creation branch from 45e0a79 to 68c2874 Compare May 29, 2026 17:03
@SC-Jerome-S

Copy link
Copy Markdown
Contributor Author

@Frzk I think I'll need your help to understand why my build fail :) I must be missing something
Thanks !

@Frzk Frzk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

suggestion: I think you could adopt a simpler way to put anchors for your titles.

suggestion: I've added a few other suggestions (abbreviations + remove deprecated addon mentions).

I've haven't found the issue with the build. The log mentions a faulty link to #scalabilité (the é is not OK, and should be replaced by e). But I haven't been able to find such a link in your PR :-/

Comment thread src/_posts/security/overview/2000-01-01-application-security.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-application-security.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-auditing.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-auditing.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-backups.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-third-parties.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-third-parties.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-connectivity.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-connectivity.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-incident-support.md Outdated

@Frzk Frzk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

suggestion: I think you could adopt a simpler way to put anchors for your titles.

suggestion: I've added a few other suggestions (abbreviations + remove deprecated addon mentions).

I've haven't found the issue with the build. The log mentions a faulty link to #scalabilité (the é is not OK, and should be replaced by e). But I haven't been able to find such a link in your PR :-/

@Frzk Frzk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you'll have to provide an id for all titles containing a special chars. (Or update Jekyll config to handle them properly - I haven't check how to do that, but there might be a way :))

Comment thread src/_posts/security/overview/2000-01-01-application-security.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-application-security.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-application-security.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-auditing.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-backups.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-business-disaster.md Outdated
Comment thread src/_posts/security/overview/2000-01-01-business-disaster.md Outdated
@SC-Jerome-S

Copy link
Copy Markdown
Contributor Author

Thanks @Frzk for your massive help! I was going round in circles. You've saved me 👍
I'll apply suggestions
PS: good catch for IPsec

Co-authored-by: François <francois+github@kubler.org>

@yanjost yanjost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The email link was removed from the English version here, but it's still present in the French translation below. This looks like an unintended regression — the EN and FR versions should be consistent.

Comment thread src/_posts/security/overview/2000-01-01-incident-support.md
Comment thread src/_posts/security/overview/2000-01-01-compliance.md
Comment thread src/_posts/security/overview/2000-01-01-business-disaster.md
- Remove support email link from FR version of incident-support (EN/FR consistency)
- Add EN abbreviation definitions (*[GDPR], *[HDS]) to compliance
- Add EN abbreviation definitions (*[BCP], *[DRP]) to business-disaster
- Update modified_at dates to 2026-06-29 for all 15 overview files

@yanjost yanjost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@SC-Jerome-S
SC-Jerome-S merged commit df9acb0 into master Jun 30, 2026
4 checks passed
@SC-Jerome-S
SC-Jerome-S deleted the feat/security-overview-FR-version-creation branch June 30, 2026 08:53
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.

3 participants