Skip to content

revert(ios): take the energy tax refund row out of the app - #149

Merged
frahlg merged 1 commit into
mainfrom
remove-battery-return
Nov 3, 2026
Merged

frahlg merged 1 commit into
mainfrom
remove-battery-return

Conversation

@frahlg

@frahlg frahlg commented Nov 3, 2026

Copy link
Copy Markdown
Member

Fredrik, 3 Nov 2026: the energy tax refund after battery storage should not be in the iPhone app. This reverts #145 (3591e80) in full: the History row, its view, model, demo answers, tests, screenshot script lines and the CLAUDE.md paragraph.

Core keeps /api/energy/battery-return and the tile on the box's own History page. Nothing on the box changes.

store/en-US/release-notes.txt still describes 0.6.6, which shipped with the row. The next version bump rewrites it.

Evidence (this Mac)

  • swift test in appleApp/FTWKit: 643 tests in 98 suites passed.
  • xcodebuild -scheme FTW for the iOS Simulator: BUILD SUCCEEDED.
  • Demo mode on iPhone 17 Pro, History tab: the list ends with Electricity price; no refund row.

🤖 Generated with Claude Code

Fredrik, 3 Nov 2026: the refund for battery storage does not belong in
the iPhone app. This reverts 3591e80 (#145). Core keeps
/api/energy/battery-return and its History tile on the box.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-03T13:38:40.773900Z 27f8dbe PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@frahlg
frahlg merged commit bf90fc0 into main Nov 3, 2026
1 check passed
@frahlg
frahlg deleted the remove-battery-return branch November 3, 2026 13:54
@frahlg frahlg mentioned this pull request Nov 3, 2026
frahlg added a commit that referenced this pull request Nov 3, 2026
Release notes in English and Swedish for #150 (the plan's clock as half
a day, the next a swipe away, and the sun as an area of the forecast)
and #149 (the energy tax refund row has left the app). The 0.6.6
paragraph no longer names the row, and the 0.6.0 one goes, so both
files stay under 4000 characters.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant