Skip to content

Fix broken Data_Protection_Policy.md links across 32 files#1086

Merged
pethers merged 2 commits intomasterfrom
copilot/fix-broken-links-data-protection
Mar 1, 2026
Merged

Fix broken Data_Protection_Policy.md links across 32 files#1086
pethers merged 2 commits intomasterfrom
copilot/fix-broken-links-data-protection

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 28, 2026

Data_Protection_Policy.md does not exist in Hack23/ISMS-PUBLIC. All 38 references across 32 files pointed to a 404. Data protection coverage in ISMS-PUBLIC is split between Privacy_Policy.md (legal/regulatory) and Data_Classification_Policy.md (technical controls).

Changes

  • 27 HTML files (blog-medical-cannabis-hipaa-gdpr, discordian-compliance, discordian-data-protection variants): replaced with Privacy_Policy.md — all are GDPR/privacy contexts
  • 3 skills files (GDPR, stakeholder-registry, supplier-management): replaced with Privacy_Policy.md
  • 2 skills files (data-classification, cryptography): In data-classification, the former Data_Protection_Policy.md entry was replaced with Privacy_Policy.md for complementary coverage (line 552 already links to Data_Classification_Policy.md as the primary policy). In cryptography, the entry was replaced with Data_Classification_Policy.md for technical data handling context.
  • Link text updated to match target policy name in English and translated versions (DE, FR, ES, NL, DA, AR)
  • Fixed duplicate in gdpr/SKILL.md where both entries became Privacy_Policy.md — second entry now references Data_Classification_Policy.md

Example

<!-- Before (404) -->
<a href="https://github.com/Hack23/ISMS-PUBLIC/blob/main/Data_Protection_Policy.md">Data Protection Policy (Public ISMS)</a>

<!-- After -->
<a href="https://github.com/Hack23/ISMS-PUBLIC/blob/main/Privacy_Policy.md">Privacy Policy (Public ISMS)</a>

Zero references to Data_Protection_Policy.md remain in the repository.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Replace non-existent Data_Protection_Policy.md with:
- Privacy_Policy.md for GDPR/privacy contexts (HTML pages, GDPR/stakeholder/supplier skills)
- Data_Classification_Policy.md for technical data handling contexts (data-classification, cryptography skills)

Update link text to match target policy names in English and translated versions.

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix broken links to data protection policy in blog posts Fix broken Data_Protection_Policy.md links across 32 files Feb 28, 2026
@sonarqubecloud
Copy link
Copy Markdown

@pethers pethers requested a review from Copilot February 28, 2026 23:59
@github-actions github-actions bot added documentation Documentation updates security Security improvements compliance Compliance and ISMS-related i18n Internationalization and localization copilot GitHub Copilot configuration skills GitHub Copilot skills content-pages Main website pages content-blog Blog content content-isms ISMS policy pages labels Mar 1, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fixes repository-wide 404s by replacing links to the non-existent Data_Protection_Policy.md in Hack23/ISMS-PUBLIC with the appropriate existing ISMS policy documents, improving link integrity across the multilingual static site and skills library.

Changes:

  • Updated GDPR/privacy-context pages to link to Privacy_Policy.md instead of Data_Protection_Policy.md.
  • Updated technical-control references (notably cryptography/GDPR skill) to link to Data_Classification_Policy.md where appropriate.
  • Updated visible link text in the affected pages/skills to match the new target policy names.

Reviewed changes

Copilot reviewed 32 out of 32 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
discordian-data-protection_de.html Replace broken ISMS policy link with Privacy_Policy.md on DE page.
discordian-data-protection_ar.html Replace broken ISMS policy link with Privacy_Policy.md on AR page.
discordian-compliance.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_ar.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_da.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_de.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_es.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_fi.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_fr.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_he.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_ja.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_ko.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_nl.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_no.html Replace broken GDPR policy reference link with Privacy_Policy.md.
discordian-compliance_zh.html Replace broken GDPR policy reference link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_ar.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_da.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_de.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_es.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_fi.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_fr.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_he.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_ja.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_ko.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_nl.html Replace broken ISMS policy link with Privacy_Policy.md.
blog-medical-cannabis-hipaa-gdpr_zh.html Replace broken ISMS policy link with Privacy_Policy.md.
.github/skills/security/data-classification/SKILL.md Update related ISMS policy link away from missing Data_Protection_Policy.md.
.github/skills/security/cryptography/SKILL.md Update related ISMS policy link to Data_Classification_Policy.md.
.github/skills/governance/supplier-management/SKILL.md Update policy filename and GitHub link to Privacy_Policy.md.
.github/skills/governance/stakeholder-registry/SKILL.md Update related documentation link to Privacy_Policy.md.
.github/skills/compliance/gdpr/SKILL.md Replace broken policy reference with Data_Classification_Policy.md (second link).

Comment on lines 552 to +553
- **[Data Classification Policy](https://github.com/Hack23/ISMS-PUBLIC/blob/main/Data_Classification_Policy.md)** - Primary policy for data classification
- **[Data Protection Policy](https://github.com/Hack23/ISMS-PUBLIC/blob/main/Data_Protection_Policy.md)** - Data protection requirements
- **[Privacy Policy](https://github.com/Hack23/ISMS-PUBLIC/blob/main/Privacy_Policy.md)** - Data protection and privacy requirements
Copy link

Copilot AI Mar 1, 2026

Choose a reason for hiding this comment

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

The PR description says the data-classification skill link was updated to point to Data_Classification_Policy.md, but this hunk updates the former Data_Protection_Policy.md reference to Privacy_Policy.md. Please either update the PR description to match what was changed here, or (if the intent was a technical-controls reference) adjust this entry to point to Data_Classification_Policy.md instead.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The code is intentionally correct as-is. Line 552 already links to Data_Classification_Policy.md as the primary policy. The former Data_Protection_Policy.md entry (line 553) was changed to Privacy_Policy.md to avoid a duplicate link and to provide complementary coverage — Privacy Policy covers the legal/regulatory side of data protection while Data Classification Policy covers the technical controls. The PR description was slightly imprecise about this specific entry.

@pethers
Copy link
Copy Markdown
Member

pethers commented Mar 1, 2026

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Mar 1, 2026

@copilot apply changes based on the comments in this thread

Reviewed the feedback. The code in data-classification/SKILL.md is correct as-is — line 552 already has Data_Classification_Policy.md as the primary policy, so line 553 intentionally uses Privacy_Policy.md for complementary coverage (legal/regulatory side) rather than creating a duplicate. The PR description was slightly imprecise about this specific entry but the links are all correct and point to existing ISMS-PUBLIC documents.

@pethers pethers marked this pull request as ready for review March 1, 2026 00:17
@pethers pethers self-requested a review as a code owner March 1, 2026 00:17
@pethers pethers merged commit 817ca45 into master Mar 1, 2026
21 checks passed
@pethers pethers deleted the copilot/fix-broken-links-data-protection branch March 1, 2026 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compliance Compliance and ISMS-related content-blog Blog content content-isms ISMS policy pages content-pages Main website pages copilot GitHub Copilot configuration documentation Documentation updates i18n Internationalization and localization security Security improvements size/M skills GitHub Copilot skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants