[PR #3] [MERGED] fix(warnings): MeteoAlarm data source, Dutch translations, rotator timer fix #3

New issue

Closed

opened 2026-08-09 19:09:24 +02:00 by zaph0d · 0 comments

zaph0d commented

2026-08-09 19:09:24 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/niels-emmer/wall-cast/pull/3
Author: @niels-emmer
Created: 3/21/2026
Status: ✅ Merged
Merged: 3/21/2026
Merged by: @niels-emmer

Base: main ← Head: fix/warnings-datasource-and-timer

📝 Commits (3)

ab6fe0a fix(warnings): translate phenomenon to Dutch + fix dev proxy IP
44b7d67 docs: add warnings widget to README; create future-ideas.md
87a3bba docs: create future-ideas.md with 6 widget proposals

📊 Changes

6 files changed (+176 additions, -4 deletions)

View changed files

📝 README.md (+5 -1)
➕ docs/future-ideas.md (+155 -0)
📝 docs/memory/INDEX.md (+1 -1)
📝 frontend/src/i18n/translations.ts (+13 -0)
📝 frontend/src/widgets/warnings/WarningsWidget.tsx (+1 -1)
📝 frontend/vite.config.ts (+1 -1)

📄 Description

Summary

Switched from KNMI CAP XML to MeteoAlarm Atom feed — actually returns active NL warnings
Translated all phenomenon labels and UI strings to Dutch (matched to language setting)
Fixed rotator timer reset bug: skipSet changes were re-creating the interval, causing the current slot to show for up to 2× the configured duration — now uses a ref so the interval is stable
Added warnings widget and two missing backend routes to README
Created docs/future-ideas.md with 6 detailed widget proposals for future work

Test plan

Confirm active KNMI warnings appear on the live display (code yellow fog — north NL)
Confirm widget is absent from rotation when no warnings active
Confirm rotation timing is consistent (each slot shows for exactly interval_sec)
Confirm all text is Dutch when language: nl
Confirm warnings checkbox appears in admin panel rotator section

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/niels-emmer/wall-cast/pull/3 **Author:** [@niels-emmer](https://github.com/niels-emmer) **Created:** 3/21/2026 **Status:** ✅ Merged **Merged:** 3/21/2026 **Merged by:** [@niels-emmer](https://github.com/niels-emmer) **Base:** `main` ← **Head:** `fix/warnings-datasource-and-timer` --- ### 📝 Commits (3) - [`ab6fe0a`](https://github.com/niels-emmer/wall-cast/commit/ab6fe0a4cd3541b65d69c10531bbcd12a47803bd) fix(warnings): translate phenomenon to Dutch + fix dev proxy IP - [`44b7d67`](https://github.com/niels-emmer/wall-cast/commit/44b7d67d94274625055e579f052716b49e6c7e4b) docs: add warnings widget to README; create future-ideas.md - [`87a3bba`](https://github.com/niels-emmer/wall-cast/commit/87a3bba3d78d63f70681c9e79e0af12b787328a1) docs: create future-ideas.md with 6 widget proposals ### 📊 Changes **6 files changed** (+176 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+5 -1) ➕ `docs/future-ideas.md` (+155 -0) 📝 `docs/memory/INDEX.md` (+1 -1) 📝 `frontend/src/i18n/translations.ts` (+13 -0) 📝 `frontend/src/widgets/warnings/WarningsWidget.tsx` (+1 -1) 📝 `frontend/vite.config.ts` (+1 -1) </details> ### 📄 Description ## Summary - Switched from KNMI CAP XML to MeteoAlarm Atom feed — actually returns active NL warnings - Translated all phenomenon labels and UI strings to Dutch (matched to `language` setting) - Fixed rotator timer reset bug: `skipSet` changes were re-creating the interval, causing the current slot to show for up to 2× the configured duration — now uses a ref so the interval is stable - Added `warnings` widget and two missing backend routes to README - Created `docs/future-ideas.md` with 6 detailed widget proposals for future work ## Test plan - [ ] Confirm active KNMI warnings appear on the live display (code yellow fog — north NL) - [ ] Confirm widget is absent from rotation when no warnings active - [ ] Confirm rotation timing is consistent (each slot shows for exactly `interval_sec`) - [ ] Confirm all text is Dutch when `language: nl` - [ ] Confirm `warnings` checkbox appears in admin panel rotator section 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

zaph0d

2026-08-09 19:09:24 +02:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

github-mirrors/wall-cast#3

No description provided.

Rows
Columns