Skip to content

Conversation

@Mips2648
Copy link
Contributor

@Mips2648 Mips2648 commented Jul 1, 2025

Description

Merge to stable last changes done by @Caelion

Suggested changelog entry

  • Ajout du type décodeur
  • Ajout de la prise en compte des différents modes sur un équipement de type Thermostat
  • Correction de bug pour PHP8
  • Correction de la gestion du MFA

Related issues/external references

Fixes #

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

  • I have checked there is no other PR open for the same change.
  • I have read the Contribution Guidelines.
  • I grant the project the right to include and distribute the code under the GNU.
  • I have added tests to cover my changes.
  • I have verified that the code complies with the projects coding standards.
  • [Required for new sniffs] I have added MD documentation for the sniff.

Jenkins and others added 19 commits November 7, 2024 00:05
Correction pour que le mode remonte correctement de Jeedom vers Google Home
Thermostat - Gestion des modes
"Modes" is neither a required nor a recommended trait for a SECURITYSYSTEM device. Different Arm Levels are handled by ArmDisarm trait.
Remove "Modes" trait from SECURITYSYSTEM device
Fix PHP warnings in jeeGshOauth.php:
```
PHP Warning:  Undefined variable $mfa in /var/www/html/plugins/gsh/core/php/jeeGshOauth.php on line 76
```
@Caelion
Copy link
Contributor

Caelion commented Sep 4, 2025

Hello @Mips2648
Do you have any news about stable version ?
Thank you,
Regards

@Mips2648 Mips2648 requested a review from zoic21 September 5, 2025 08:55
@Mips2648
Copy link
Contributor Author

Mips2648 commented Sep 5, 2025

Hi @Caelion
I'm not allowed to merge to stable, so I cannot do anything more on this topic

@Caelion
Copy link
Contributor

Caelion commented Sep 25, 2025

Hello
Ok thanks
@zoic21 : could you do something ?
Thank you

@zoic21
Copy link
Contributor

zoic21 commented Sep 26, 2025

Hello,
Sorry for the delay, it's approved on my side

@Caelion
Copy link
Contributor

Caelion commented Sep 26, 2025

Thanks zoic

Hello @Salvialf
Are you the correct person to merge it ?
Thank you
Regards

@Salvialf
Copy link

Hello @Salvialf Are you the correct person to merge it ? Thank you Regards

Hi,
I’m not familiar with Google Smart Home, so if multiple users confirm it works, that’s fine by me.

@zoic21 you don’t need to wait for more reviews — if you approve, feel free to merge it right away.

@zoic21 zoic21 merged commit 1b87db0 into master Sep 27, 2025
6 of 8 checks passed
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.

8 participants