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.

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

80 bug data for key data does not exist #96

Merged
merged 24 commits into from
Dec 3, 2024

Conversation

jontofront
Copy link
Owner

This pull request includes various changes to the econet300 integration, focusing on refactoring the API methods, improving logging, and updating constant values. The most important changes include renaming methods for clarity, adding new API endpoints, and enhancing the logging and error handling mechanisms.

Refactoring and Method Renaming:

  • Renamed get_params to fetch_sys_params to better describe its functionality and added more detailed logging. [1] [2]
  • Renamed fetch_data to fetch_reg_params_data and added error handling for data fetching.

Logging Improvements:

  • Added detailed logging in methods to provide better insights into the execution flow and data fetching processes. [1] [2]

Constants and Configuration:

  • Introduced new constants for sensor and binary sensor mappings, and updated existing ones for better clarity and usage. [1] [2]
  • Updated .vscode/settings.json to include new words in the spell checker configuration.

Error Handling:

  • Enhanced error handling in data fetching methods to log errors and handle missing data gracefully. [1] [2]

Code Cleanup:

  • Fixed typos and removed redundant debug statements to clean up the codebase. [1] [2] [3]

@jontofront jontofront linked an issue Dec 3, 2024 that may be closed by this pull request
@jontofront jontofront merged commit 34c8cb6 into master Dec 3, 2024
6 checks passed
@jontofront jontofront deleted the 80-bug-data-for-key-data-does-not-exist branch December 12, 2024 07:51
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.

[Bug]: Data for key: data does not exist
1 participant