webui: fix line break in the English access point update button label - #81
Conversation
The English label of the button for updating Homematic IP access points with incompatible firmware broke the line after "with", which split the phrase "with incompatible FW" across both lines. Move the line break before "with" so the label reads "Update Access Points" on the first and "with incompatible FW" on the second line. This natively replaces OpenCCU rootfs patch 0128-WebUI-Fix-UpdateAccessPointButton.patch.
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe English ChangesAccess point label wrapping
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The change only adjusts the English label’s wrapping, with no identified user-facing or operational issue. It is ready to merge. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Original OpenCCU rootfs patch: 0128
Original filename:
0128-WebUI-Fix-UpdateAccessPointButton.patchOriginal patch:
''https://github.com/OpenCCU/OpenCCU/blob/f795bc0418e10ca244cbe8ca7893e835d9523152/buildroot-external/package/openccu-base/rootfs-patches/0128-WebUI-Fix-UpdateAccessPointButton.patch''The English label of the button for updating Homematic IP access points with incompatible firmware (
btnAccessPointsinsrc/webui/www/webui/js/lang/en/translate.lang.extension.js) broke the line after "with". This split the phrase "with incompatible FW" across both lines. This moves the line break before "with", so the label reads "Update Access Points" on the first line and "with incompatible FW" on the second. The German label is unchanged.OpenCCU rootfs patches 0001, 0007, 0044 and 0099 also modify this file and precede 0128 in the series. None of their hunks covers this line, and the line count is unchanged, so they all still apply unchanged.
Validation
scripts/base-patch-migration.py validatewith the canonical Buildroot 2026.08git4archive of this branch (5c691705, sha256ef08fbb8d017004e8fcbcad13874ed84cefde620e5c6be967af6e217c395c4f4) and--skip-patch 0128: PASS (the other 122 rootfs patches apply with zero fuzz).compareagainst the current pin baseline (2aee789d, all 123 patches): IDENTICAL (4668 entries).OpenCCU cleanup PR (patch removal, pin and archive hash update): OpenCCU/OpenCCU#4275
Summary by CodeRabbit