New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Store preferences in disk file on host platform #6428
Conversation
30 chars get used very quickly when the user uses a few special characters in the version string. Eg. `Gázkazán 1-2-4 PID vezérlés` requires double amount of chars: `G\303\241zkaz\303\241n 1-2-4 PID vez\303\251rl\303\251s`.
Use hash to compare project versions.
Hey there @esphome/core, mind taking a look at this pull request as it has been labeled with an integration ( |
fs::create_directories(this->filename_); | ||
this->filename_.append("/"); | ||
this->filename_.append(App.get_name()); | ||
this->filename_.append(".json"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on the below code, this file is binary, not json?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, yes. JSON seemed like a Good Idea (which in Python it would have been) then I decided that was over-engineering.
What does this implement/fix?
Types of changes
Related issue or feature (if applicable): fixes
Pull request in esphome-docs with documentation (if applicable): esphome/esphome-docs#
Test Environment
Example entry for
config.yaml
:# Example config.yaml
Checklist:
tests/
folder).If user exposed functionality or configuration variables are added/changed: