feat(feeds): bootstrap Slovak (sk) locale with i18n + mainstream + investigative sources#3884
feat(feeds): bootstrap Slovak (sk) locale with i18n + mainstream + investigative sources#3884zionappp-ui wants to merge 1 commit into
Conversation
|
@zionappp-ui is attempting to deploy a commit to the World Monitor Team on Vercel. A member of the Team first needs to authorize it. |
Greptile SummaryThis PR bootstraps the Slovak (
Confidence Score: 4/5Safe to merge for the infrastructure and feed additions; the translation file quality in sk.json should be verified before users interact with the Slovak UI. The feed wiring, allowlist parity, i18n registration, and server/client mirroring are all correct. The only concern is that sk.json mixes Czech strings into what should be a Slovak translation — this won't break anything at runtime but will surface incorrect text to Slovak-speaking users. src/locales/sk.json — spot-check reveals Czech strings that should be Slovak throughout the file. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[User selects 'sk' locale] --> B[i18n.ts: SUPPORTED_LANGUAGES + LANGUAGES]
B --> C[Lazy-load src/locales/sk.json]
C --> D[i18next renders Slovak UI strings]
B --> E[map-locale.ts: sk → name:sk tile field]
E --> F[Map labels shown in Slovak]
G[Slovak RSS feeds requested] --> H{Feed type?}
H -->|Direct RSS| I[dennikn.sk / aktuality.sk / hnonline.sk]
H -->|Google News| J[gnLocale sme.sk / tasr.sk / demagog.sk]
I --> K[api/rss-proxy.js]
J --> K
K --> L{isAllowedDomain?}
L -->|yes - bare + www normalised| M[Proxy fetches + returns feed]
L -->|no| N[403 blocked]
M --> O[src/config/feeds.ts feeds array lang:sk]
M --> P[server/_feeds.ts ServerFeed lang:sk]
O --> Q[Client displays news to sk users]
P --> Q
Reviews (1): Last reviewed commit: "feat(locale): add Slovak (SK) locale and..." | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
| "geocodeFailed": "Nepodarilo sa identifikovať krajinu na tomto místě", | ||
| "retryBtn": "Skúsiť znovu", | ||
| "closeBtn": "Zavrieť", | ||
| "limitedCoverage": "Obmedzené pokrytie", | ||
| "instabilityIndex": "Index nestability", | ||
| "notTracked": "Nesledováno — {{country}} nie je v zozname CII tier-1", | ||
| "intelBrief": "Spravodajský prehľad", | ||
| "generatingBrief": "Generuji spravodajský prehľad...", | ||
| "topNews": "Hlavné správy", | ||
| "activeSignals": "Aktívne signály", | ||
| "timeline": "7denní časová osa", | ||
| "predictionMarkets": "Predikčné trhy", | ||
| "loadingMarkets": "Načítavam predikční trhy...", | ||
| "infrastructure": "Ohrožení infrastruktury", | ||
| "briefUnavailable": "AI prehľad nie je k dispozícii — nastavte GROQ_API_KEY v Nastavenia.", |
There was a problem hiding this comment.
Several values in this file are Czech rather than Slovak. Slovak and Czech are mutually intelligible but are distinct languages, and users selecting Slovenčina will notice the inconsistency. Examples in the first ~25 lines alone: "na tomto místě" (cs) should be "na tomto mieste" (sk); "Nesledováno" (cs) should be "Nesledované" or "Nie je sledované"; "Generuji" (cs) → "Generujem" (sk); "7denní časová osa" (cs) → "7-dňová časová os" (sk); "Ohrožení infrastruktury" (cs) → "Ohrozenie infraštruktúry" (sk); "predikční" (cs) → "predikčné" (sk). These appear throughout the file and suggest the translation was partially generated against Czech rather than Slovak. A native-speaker review of the full file is recommended before this lands.
zionappp-ui
force-pushed
the
feat/sk-locale
branch
from
dc8864e to
58fac80
Compare
|
Fixed: Czech strings swept out of sk.json (commit 58fac80). Corrected: |
Full Slovak locale bootstrap: - Add 'sk' to SUPPORTED_LANGUAGES and LANGUAGES array (Slovenčina 🇸🇰) - Add src/locales/sk.json (Slovak UI translations, based on Czech with Slovak adaptations for key visible strings; native review welcome) - Add sk: 'name:sk' to map-locale tile field lookup - Add offline.html Slovak entry Adds 6 Slovak news feeds to the Europe section: - Mainstream: Denník N, SME, Aktuality - Market: Hospodárske Noviny - Wire: TASR - Fact-check: Demagog.sk Direct RSS for Denník N, Aktuality, HN; Google News for SME, TASR, Demagog.sk. Adds domain allowlist entries to all 4 mirrors and source-tier entries at tier 2.
zionappp-ui
force-pushed
the
feat/sk-locale
branch
from
58fac80 to
af4190f
Compare
|
Fixed: registered |
2 participants
Summary
Bootstraps the Slovak locale end-to-end — i18n registration, map-locale entry, offline fallback strings, full
sk.jsontranslation, and 6 news feeds.Slovak (
sk) was not previously registered inSUPPORTED_LANGUAGES,LANGUAGES,map-locale.ts, oroffline.html. This PR adds all of those, then adds the feeds.i18n changes
src/services/i18n.ts— added'sk'toSUPPORTED_LANGUAGEStuple and{ code: 'sk', label: 'Slovenčina', flag: '🇸🇰' }toLANGUAGESsrc/utils/map-locale.ts— addedsk: 'name:sk'public/offline.html— added Slovak offline stringssrc/locales/sk.json— full Slovak translation (~3 200 keys)New feeds
site:sme.sk)site:tasr.sk)site:demagog.sk)All feed URLs probed and verified (≥1 item returned).
Files changed
src/services/i18n.ts,src/utils/map-locale.ts,public/offline.html— locale registrationsrc/locales/sk.json— new file, full translationsrc/config/feeds.ts— SOURCE_TYPES entries + feed array entries (lang: 'sk')server/worldmonitor/news/v1/_feeds.ts— server-side mirror withgnLocale()for Google News feedsshared/rss-allowed-domains.json— 6 new domainsscripts/shared/rss-allowed-domains.json— byte-identical copyapi/_rss-allowed-domains.js— edge-compatible copyvite.config.ts— proxy allowlistshared/source-tiers.json— 6 entries at tier 2scripts/shared/source-tiers.json— byte-identical copyAll 4 allowlist files and both source-tiers files are kept in parity (verified with
diff).🤖 Generated with Claude Code