Skip to content

[RF] Refactor RF Configuration Management#2245

Merged
1technophile merged 1 commit into1technophile:developmentfrom
Odyno:feature/rf_config_refactory
Dec 7, 2025
Merged

[RF] Refactor RF Configuration Management#2245
1technophile merged 1 commit into1technophile:developmentfrom
Odyno:feature/rf_config_refactory

Conversation

@Odyno
Copy link
Contributor

@Odyno Odyno commented Oct 17, 2025

This is the same code present on #2181, I have just renamed the branch and squashed the commit for better comprehension of content on my repository.

Description:

  • Introduced RFConfiguration class to encapsulate RF settings and operations.
  • Replaced direct usage of RFConfig structure with iRFConfig instance across multiple files.
  • Updated frequency handling in actuatorSomfy, gatewayPilight, gatewayRF, and gatewayRF2 to use iRFConfig.
  • Modified webUI to interact with iRFConfig for RF settings management.
  • Removed deprecated RFConfig structure and related functions.
  • Enhanced JSON handling for RF configuration loading and saving.
  • Improved logging for RF configuration operations.

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

- Introduced RFConfiguration class to encapsulate RF settings and operations.
- Replaced direct usage of RFConfig structure with iRFConfig instance across multiple files.
- Updated frequency handling in actuatorSomfy, gatewayPilight, gatewayRF, and gatewayRF2 to use iRFConfig.
- Modified webUI to interact with iRFConfig for RF settings management.
- Removed deprecated RFConfig structure and related functions.
- Enhanced JSON handling for RF configuration loading and saving.
- Improved logging for RF configuration operations.
@1technophile
Copy link
Owner

Thanks

@1technophile 1technophile merged commit c6b2aae into 1technophile:development Dec 7, 2025
80 checks passed
@Odyno Odyno deleted the feature/rf_config_refactory branch December 27, 2025 15:01
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.

2 participants