Fix broken Data_Protection_Policy.md links across 32 files#1086
Fix broken Data_Protection_Policy.md links across 32 files#1086
Conversation
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>
|
There was a problem hiding this comment.
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.mdinstead ofData_Protection_Policy.md. - Updated technical-control references (notably cryptography/GDPR skill) to link to
Data_Classification_Policy.mdwhere 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). |
| - **[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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
|
@copilot apply changes based on the comments in this thread |
Reviewed the feedback. The code in |



Data_Protection_Policy.mddoes not exist inHack23/ISMS-PUBLIC. All 38 references across 32 files pointed to a 404. Data protection coverage in ISMS-PUBLIC is split betweenPrivacy_Policy.md(legal/regulatory) andData_Classification_Policy.md(technical controls).Changes
Privacy_Policy.md— all are GDPR/privacy contextsPrivacy_Policy.mdData_Protection_Policy.mdentry was replaced withPrivacy_Policy.mdfor complementary coverage (line 552 already links toData_Classification_Policy.mdas the primary policy). In cryptography, the entry was replaced withData_Classification_Policy.mdfor technical data handling context.gdpr/SKILL.mdwhere both entries becamePrivacy_Policy.md— second entry now referencesData_Classification_Policy.mdExample
Zero references to
Data_Protection_Policy.mdremain 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.