Conversation
…MS documentation links Co-authored-by: pethers <[email protected]>
There was a problem hiding this comment.
Pull request overview
This PR introduces three new HTML pages for the Riksdagsmonitor Swedish Parliament Intelligence Platform, establishing its web presence on hack23.com following established patterns from existing project pages. The pages provide comprehensive project information, detailed feature descriptions, and complete ISMS documentation links.
Changes:
- Main project page (
riksdagsmonitor.html) with hero section, feature overview, platform capabilities, security badges, use cases, and ISMS compliance summary - Detailed features page (
riksdagsmonitor-features.html) covering parliamentary monitoring, historical data, multi-language support, analytics, data quality, and security compliance - Documentation hub (
riksdagsmonitor-docs.html) with 26 ISMS documentation cards linking to architecture diagrams, security design, testing frameworks, API docs, and compliance assessments
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| riksdagsmonitor.html | Main landing page with project overview, key features grid, platform capabilities, security badges, use cases, and ISMS compliance section |
| riksdagsmonitor-features.html | Comprehensive feature descriptions organized into sections for parliamentary monitoring, historical data, language support, analytics, data quality, and security compliance |
| riksdagsmonitor-docs.html | Architecture documentation hub with 26 categorized documentation cards covering current/future architecture, security, testing, API, compliance, and business continuity |
riksdagsmonitor.html
Outdated
| <link rel="stylesheet" type="text/css" href="styles.css"> <link rel="canonical" href="https://hack23.com/riksdagsmonitor.html"> | ||
|
|
There was a problem hiding this comment.
Missing newline between stylesheet and canonical link tags. This creates a readability issue and deviates from the formatting pattern used throughout the rest of the file. Add a newline after the closing > of the stylesheet link tag.
| <link rel="stylesheet" type="text/css" href="styles.css"> <link rel="canonical" href="https://hack23.com/riksdagsmonitor.html"> | |
| <link rel="stylesheet" type="text/css" href="styles.css"> | |
| <link rel="canonical" href="https://hack23.com/riksdagsmonitor.html"> |
There was a problem hiding this comment.
Fixed in ccf9072 — added newline between the stylesheet and canonical link tags in all three files.
riksdagsmonitor-features.html
Outdated
| <meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
| <meta name="description" content="Riksdagsmonitor features: Swedish Parliament monitoring, voting records, 50+ years historical data, 14 language support, political analytics, and transparency metrics."> | ||
| <title>Riksdagsmonitor Features | Swedish Parliament Intelligence | Hack23</title> | ||
| <link rel="stylesheet" type="text/css" href="styles.css"><link rel="canonical" href="https://hack23.com/riksdagsmonitor-features.html"> |
There was a problem hiding this comment.
Missing newline between stylesheet and canonical link tags. Add a newline after the closing > of the stylesheet link tag for consistency with formatting elsewhere in the file.
| <link rel="stylesheet" type="text/css" href="styles.css"><link rel="canonical" href="https://hack23.com/riksdagsmonitor-features.html"> | |
| <link rel="stylesheet" type="text/css" href="styles.css"> | |
| <link rel="canonical" href="https://hack23.com/riksdagsmonitor-features.html"> |
riksdagsmonitor-docs.html
Outdated
| href="https://fonts.googleapis.com/css2?family=Inter:wght@400;500;600;700&family=Orbitron:wght@400;500;600;700&family=Share+Tech+Mono&display=swap" | ||
| rel="stylesheet" | ||
| /> | ||
| <link rel="stylesheet" type="text/css" href="styles.css"><link rel="canonical" href="https://hack23.com/riksdagsmonitor-docs.html" /> |
There was a problem hiding this comment.
Missing newline between stylesheet and canonical link tags. Add a newline after the closing > of the stylesheet link tag for consistency with formatting elsewhere in the file.
| <link rel="stylesheet" type="text/css" href="styles.css"><link rel="canonical" href="https://hack23.com/riksdagsmonitor-docs.html" /> | |
| <link rel="stylesheet" type="text/css" href="styles.css"> | |
| <link rel="canonical" href="https://hack23.com/riksdagsmonitor-docs.html" /> |
|
@copilot apply changes based on the comments in this thread |
…ksdagsmonitor pages Co-authored-by: pethers <[email protected]>
| <meta property="og:locale" content="en_US"> | ||
| <meta property="og:locale:alternate" content="sv_SE"> |
There was a problem hiding this comment.
Missing comprehensive og:locale:alternate tags. Only sv_SE is listed as an alternate locale, but the platform emphasizes 14 language support. The established pattern in documentation pages (cia-docs.html, cia-compliance-manager-docs.html) includes all 14 language locales (ar_SA, da_DK, de_DE, es_ES, fi_FI, fr_FR, he_IL, ja_JP, ko_KR, nl_NL, nb_NO, sv_SE, zh_CN). Add all missing og:locale:alternate tags to match the pattern and support the claimed multi-language accessibility.
| <link rel="alternate" hreflang="en" href="https://hack23.com/riksdagsmonitor-docs.html"> | ||
| <link rel="alternate" hreflang="x-default" href="https://hack23.com/riksdagsmonitor-docs.html"> |
There was a problem hiding this comment.
Incomplete hreflang implementation. Only two hreflang tags are present (en and x-default), while existing documentation pages follow a comprehensive pattern. Compare with cia-docs.html and cia-compliance-manager-docs.html which include hreflang tags for all 14 languages with proper regional variants (ar, ar-SA, ar-EG, da, de, de-DE, en, es, es-ES, fi, fr, fr-FR, he, he-IL, ja, ja-JP, ko, ko-KR, nl, nl-NL, no, nb, sv, sv-SE, zh, zh-CN, zh-SG, zh-Hans). Add complete hreflang tags to ensure proper SEO and international accessibility.
| <meta property="og:locale" content="en_US"> | ||
| <meta property="og:locale:alternate" content="sv_SE"> |
There was a problem hiding this comment.
Missing comprehensive og:locale:alternate tags for all supported languages. The PR description states the platform supports 14 languages (Swedish, English, Arabic, Chinese, Danish, Dutch, Finnish, French, German, Hebrew, Japanese, Korean, Norwegian, and Spanish). However, only sv_SE is listed as an alternate locale. According to the established pattern in existing project pages (cia-project.html and compliance-manager.html), all supported language variants should be included with their corresponding locale codes. Add the missing og:locale:alternate tags for all 14 supported languages.
There was a problem hiding this comment.
Fixed in 24030aa — added all 13 og:locale:alternate tags (ar_SA, da_DK, de_DE, es_ES, fi_FI, fr_FR, he_IL, ja_JP, ko_KR, nl_NL, nb_NO, sv_SE, zh_CN) to riksdagsmonitor.html, riksdagsmonitor-features.html, and riksdagsmonitor-docs.html.
| <link rel="alternate" hreflang="en" href="https://hack23.com/riksdagsmonitor.html"> | ||
| <link rel="alternate" hreflang="x-default" href="https://hack23.com/riksdagsmonitor.html"> |
There was a problem hiding this comment.
Incomplete hreflang implementation. According to stored memory "HTML pages are expected to include hreflang tags with x-default and a self-reference", and the PR description states 14 language support. However, only two hreflang tags are present (en and x-default). Existing project pages like cia-project.html include comprehensive hreflang tags for all 14 supported languages with proper regional variants. Add hreflang tags for all 14 languages following the established pattern.
There was a problem hiding this comment.
Fixed in 24030aa — added complete hreflang tags for all 14 languages with regional variants (ar/ar-SA/ar-EG, da, de/de-DE, en, es/es-ES, fi, fr/fr-FR, he/he-IL, ja/ja-JP, ko/ko-KR, nl/nl-NL, no/nb, sv/sv-SE, zh/zh-CN/zh-SG/zh-Hans, x-default) to all three riksdagsmonitor pages.
| <meta property="og:locale" content="en_US"> | ||
| <meta property="og:locale:alternate" content="sv_SE"> |
There was a problem hiding this comment.
Missing comprehensive og:locale:alternate tags. The PR description and content emphasize 14 language support, but only sv_SE is listed as an alternate locale. Existing project pages (cia-project.html, compliance-manager.html) include all 14 language locales. Add og:locale:alternate tags for ar_SA, da_DK, de_DE, es_ES, fi_FI, fr_FR, he_IL, ja_JP, ko_KR, nl_NL, nb_NO, and zh_CN to match the established pattern and the claimed multi-language support.
| <link rel="alternate" hreflang="en" href="https://hack23.com/riksdagsmonitor-features.html"> | ||
| <link rel="alternate" hreflang="x-default" href="https://hack23.com/riksdagsmonitor-features.html"> |
There was a problem hiding this comment.
Incomplete hreflang implementation. Only two hreflang tags are present (en and x-default), but the platform claims 14 language support. Existing feature pages like cia-features.html and cia-compliance-manager-features.html include comprehensive hreflang tags for all supported languages with proper regional variants (ar, ar-SA, da, de, de-DE, en, es, es-ES, fi, fr, fr-FR, he, he-IL, ja, ja-JP, ko, ko-KR, nl, nl-NL, no, nb, sv, sv-SE, zh, zh-CN, etc.). Add complete hreflang tags for consistency with established patterns and SEO best practices.
|
@copilot apply changes based on the comments in this thread |
…es to riksdagsmonitor pages Co-authored-by: pethers <[email protected]>
…cs/ directory Co-authored-by: pethers <[email protected]>
github-actions
bot
added
infrastructure
Signed-off-by: James Pether Sörling <[email protected]>
|
#1069/#1070 already merged) Co-authored-by: pethers <[email protected]>
Establishes hack23.com presence for the Riksdagsmonitor Swedish Parliament Intelligence Platform with three new pages following established Hack23 project page patterns.
New Pages
riksdagsmonitor.html— Main project page: hero, OpenSSF/CodeQL/Release badge strip, key features grid, platform capabilities, security section, ISMS compliance summary, CTAriksdagsmonitor-features.html— Parliamentary monitoring, 50+ years historical data, 14-language support, political behavior analytics, automated data validation, digital signaturesriksdagsmonitor-docs.html— 26 ISMS documentation cards covering architecture, security, state diagrams, flowcharts, data models, testing (unit/coverage/E2E/performance), API, BCP, threat model, CRA assessmentSEO & Internationalization
All three pages include the full 14-language suite matching existing project pages:
og:locale:alternatefor all 14 locales (ar_SA,da_DK,de_DE,es_ES,fi_FI,fr_FR,he_IL,ja_JP,ko_KR,nl_NL,nb_NO,sv_SE,zh_CN)hreflangtags with regional variants (ar,ar-SA,da,de,de-DE,en,es,es-ES,fi,fr,fr-FR,he,he-IL,ja,ja-JP,ko,ko-KR,nl,nl-NL,no,nb,sv,sv-SE,zh,zh-CN,zh-SG,zh-Hans,x-default)CI Fix
html5validatorstep inpullrequest.ymlupdated to--blacklist docsto exclude auto-generated report files indocs/from HTML validation.Original prompt
This section details on the original issue you should resolve
<issue_title>Add Riksdagsmonitor project pages (main, features, docs) with full ISMS documentation links</issue_title>
<issue_description>## 🎯 Objective
Create three new HTML pages for the Riksdagsmonitor project following the same pattern as existing Hack23 project pages (
black-trigram.html,compliance-manager.html,cia-project.html), including all ISMS documentation links.📋 Background
Riksdagsmonitor is a Swedish Parliament Intelligence Platform that monitors political activity with systematic transparency. Built on the Citizen Intelligence Agency (CIA) platform, it provides 50+ years of historical data, real-time analysis, and supports 14 languages. It is fully ISMS-compliant with a comprehensive security architecture.
This project needs homepage presence with three pages:
riksdagsmonitor.html— Main project page (overview, features, links)riksdagsmonitor-features.html— Detailed feature descriptionsriksdagsmonitor-docs.html— Architecture & ISMS documentation hub📊 Current State
✅ Acceptance Criteria
riksdagsmonitor.htmlcreated followingcia-project.htmlpatternriksdagsmonitor-features.htmlcreated followingcia-features.htmlpatternriksdagsmonitor-docs.htmlcreated followingcia-compliance-manager-docs.htmlpattern with ALL documentation links<head>meta tags (description, keywords, OG, hreflang, canonical, schema.org)styles.csswithout modification🛠️ Implementation Guidance
Files to Create
riksdagsmonitor.html— Copy structure fromcia-project.html:riksdagsmonitor-features.html— Copy structure fromcia-features.html:riksdagsmonitor-docs.html— Copy structure fromcia-compliance-manager-docs.htmlwith these doc links:Documentation Cards for
riksdagsmonitor-docs.htmlAll links follow pattern
https://github.com/Hack23/riksdagsmonitor/blob/master/:ARCHITECTURE.mdFUTURE_ARCHITECTURE.mdSTATEDIAGRAM.mdFUTURE_STATEDIAGRAM.mdFLOWCHART.mdFUTURE_FLOWCHART.mdSECURITY_ARCHITECTURE.mdFUTURE_SECURITY_ARCHITECTURE.mdSWOT.mdFUTURE_SWOT.mdWORKFLOWS.mdFUTURE_WORKFLOWS.mdMINDMAP.mdFUTURE_MINDMAP.mdDATA_MODEL.mdFUTURE_DATA_MODEL.mdhttps://riksdagsmonitor.com/docs/test-results// Test Plan:UnitTestPlan.mdhttps://riksdagsmonitor.com/docs/coverage/https://riksdagsmonitor.com/docs/cypress// Plan:E2ETestPlan.mdperformance-testing.mdhttps://riksdagsmonitor.com/docs/api/BCPPlan.mdEnd-of-Life-Strategy.mdFinancialSecurityPlan.mdTHREAT_MODEL.mdCRA-ASSESSMENT.mdKey Project Details for Content
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.