feat(charging): sync Supercharger costs from Tesla history - #5634
Conversation
✅ Deploy Preview for teslamate ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Thanks for your contribution JOUET. This is a much aksed feature. Using an AI assistant is fine — but please review and run its output yourself before submitting, and be transparent about it in the PR description (e.g. a short "tests drafted with , reviewed and verified by me" note). Reviewing code the author hasn't run or reviewed shifts the verification cost onto the maintainers. If the CI fails in the first verification step this is a big hint, nothing has been checked on your end. Please see |
|
Yes, you're right, I was too quick to open the PR. This feature started as a side project I originally wrote in Python, running in a separate Docker container alongside TeslaMate. I asked GPT to port that code directly into the TeslaMate codebase, and I submitted the PR without running the CI locally first. I'll take some time to properly review the generated changes and fix the CI issues. At first glance, it looks like it's mainly linting, but I'll go through everything before pushing further changes. I'll also update the PR description to be transparent about the use of AI. Thanks for pointing this out. |
Summary
The feature is opt-in with:
ENABLE_TESLA_CHARGING_COST_SYNC=trueThis addresses the feature discussed in:
#1831
#3266
Testing
AI assistance
Parts of this implementation and its tests were drafted with AI assistance, then reviewed and verified by me.