Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Implement custom fields for power profiles #2873

Merged
merged 20 commits into from
Jan 1, 2025
Merged

Implement custom fields for power profiles #2873

merged 20 commits into from
Jan 1, 2025

Conversation

bramstroker
Copy link
Owner

@bramstroker bramstroker commented Dec 31, 2024 •
edited
Loading

  • More tests. number selector, validator
  • Documentation
  • Forward to post library flow?
  • JSON schema extend (number field)
  • Check: custom_components/powercalc/config_flow.py:941
  • Field translations. How?
  • Validator. check if all variables have been passed for all custom fields
  • YAML support
  • Check if replacements in templates work. Also modify docs
  • Rename name to label
  • YAML empty config or include config should not discover profiles with custom fields. Only write tests
  • Rename custom_fields to variables in config?

@github-actions github-actions bot added the enhancement New feature or request label Dec 31, 2024
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 1, 2025
bramstroker and others added 16 commits January 1, 2025 12:18
@bramstroker bramstroker merged commit 9f8c3a4 into master Jan 1, 2025
18 checks passed
@bramstroker bramstroker deleted the feat/profile-fields branch January 5, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation enhancement New feature or request powerprofile
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bramstroker