admin
0 Followers · 0 Following
  • Joined on 2025-08-30
Repositories
5
 Projects Packages Public Activity Starred Repositories
admin pushed to release/1.0 at admin/goodwe-addon 2026-10-22 18:57:39 +02:00
aa228062c5 Merge PR #206: PRICE-02 the price verdict skips ended periods
6b2d0091d6 PRICE-02 review: keep an open buy over midnight in the real-year replay; say which variant the kWh figure is
3dd4dd89e0 PRICE-02 round 3: say that hourly and static plans change too, test plans built off the hour, exact period end and the Home page path, production-shaped real-year replay
891ed7bdd7 PRICE-02 round 2: require now, undo the test loosening, analyse ties
2ff9be710c PRICE-02: correct the DECISIONS note (which sentences changed, how identity is pinned)
Compare 7 commits »
admin merged pull request admin/goodwe-addon#206 2026-10-22 18:57:38 +02:00
PRICE-02: the trading verdict skips price periods that have ended
admin commented on pull request admin/goodwe-addon#206 2026-10-22 17:38:29 +02:00
PRICE-02: the trading verdict skips price periods that have ended

Correction to my comment above: not merged. My merge call was refused by a permission check on my side, so I stopped and did not try another way. The review verdict stands: ready to merge at…

admin commented on pull request admin/goodwe-addon#206 2026-10-22 17:37:09 +02:00
PRICE-02: the trading verdict skips price periods that have ended

PRICE-02 re-review (round 3, head 3dd4dd8)

Reviewed 3dd4dd89e025a821e854e853e182942061e96d1e (Router T93 PASS). I pushed one test-and-text commit on top: `6b2d0091d6ead5b376b80d0afa654e3e5d50…

admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 16:48:45 +02:00
6b2d0091d6 PRICE-02 review: keep an open buy over midnight in the real-year replay; say which variant the kWh figure is
admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 14:59:15 +02:00
3dd4dd89e0 PRICE-02 round 3: say that hourly and static plans change too, test plans built off the hour, exact period end and the Home page path, production-shaped real-year replay
admin commented on pull request admin/goodwe-addon#206 2026-10-22 12:27:30 +02:00
PRICE-02: the trading verdict skips price periods that have ended

PRICE-02 review (head 891ed7bdd760f43a8c05e89090fb5d37821a6262): not merged, owner decision needed

The code fix is correct and sits in one place. It is not merged because of one undisclosed…

admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 10:37:45 +02:00
891ed7bdd7 PRICE-02 round 2: require now, undo the test loosening, analyse ties
admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 08:37:43 +02:00
2ff9be710c PRICE-02: correct the DECISIONS note (which sentences changed, how identity is pinned)
admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 08:20:53 +02:00
ecd9822861 PRICE-02: pin the modes-off identity with a digest that hides only the changed words
admin created pull request admin/goodwe-addon#206 2026-10-22 07:19:37 +02:00
PRICE-02: the trading verdict skips price periods that have ended
admin created branch PRICE-02 in admin/goodwe-addon 2026-10-22 07:19:29 +02:00
admin pushed to PRICE-02 at admin/goodwe-addon 2026-10-22 07:19:29 +02:00
4297f7f3f1 PRICE-02: the trading verdict skips price periods that have ended
admin pushed to release/1.0 at admin/goodwe-addon 2026-10-22 06:10:38 +02:00
90bc9017b2 Merge pull request 'UX-46: Dutch status rows, banners, notes and wizard lines on NL pages; peak forecast label without (W)' (#205) from UX-46 into release/1.0
ae0ef4296b UX-46 review: Dutch wording fixes on the NL rows and notes
c1809b4ca9 UX-46: test_ux46 points at MUTATION.md and reads rows without raising
7a392a5367 UX-46: Dutch status rows, banners, notes and wizard lines on the NL pages
Compare 4 commits »
admin merged pull request admin/goodwe-addon#205 2026-10-22 06:10:37 +02:00
UX-46: Dutch status rows, banners, notes and wizard lines on NL pages; peak forecast label without (W)
admin commented on pull request admin/goodwe-addon#205 2026-10-22 05:43:07 +02:00
UX-46: Dutch status rows, banners, notes and wizard lines on NL pages; peak forecast label without (W)

UX-46 review (Opus), reviewed c1809b4, then pushed text-only fixes as ae0ef42.

No behaviour change: confirmed. Demo base e077ae7 vs head, same fixtures: /status?lang=en and /status…

admin pushed to UX-46 at admin/goodwe-addon 2026-10-22 04:56:40 +02:00
ae0ef4296b UX-46 review: Dutch wording fixes on the NL rows and notes
admin created pull request admin/goodwe-addon#205 2026-10-22 03:41:30 +02:00
UX-46: Dutch status rows, banners, notes and wizard lines on NL pages; peak forecast label without (W)
admin pushed to UX-46 at admin/goodwe-addon 2026-10-22 03:41:21 +02:00
c1809b4ca9 UX-46: test_ux46 points at MUTATION.md and reads rows without raising
7a392a5367 UX-46: Dutch status rows, banners, notes and wizard lines on the NL pages
Compare 2 commits »
admin created branch UX-46 in admin/goodwe-addon 2026-10-22 03:41:21 +02:00