Repository navigation
Suites de relecture : couleur sombre, volume 0, docs du bouton et du délai - #42
Merged
Merged
Conversation
…délai Remarques faites sur lfpoulain#26, lfpoulain#34, lfpoulain#36 et lfpoulain#38 (fusionnées dans 0.3.0) : - lfpoulain#26 : la couleur demandée et la couleur relue sont comparées à pleine intensité (Home Assistant relit [128, 0, 128] en [255, 0, 255]) ; le faux Home Assistant des tests relit comme lui ; test de la route admin POST /control avec rgbColor ; architecture.md cite couleur et blanc. - lfpoulain#34 : confidentialite.md, le bouton coupe le micro en veille et ferme la conversation pendant une conversation. - lfpoulain#36 : le délai propre au satellite est pris à satelliteWake, un changement vaut pour la conversation suivante (satellites.md, administration.md). - lfpoulain#38 : sonneries et voix partagent le volume du codec (board_set_volume → esp_codec_dev_set_out_vol) : au volume 0, l'outil le rappelle et la doc le dit ; la fiche du satellite suit un réglage vocal (resynchronisée sur la liste rechargée hors réglage en cours). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Nov 9, 2026
Owner
|
Merci, fusionnée dans |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
But
Reprendre les remarques non bloquantes faites sur #26, #34, #36 et #38, fusionnées dans 0.3.0.
Ce qui change
#26, couleur des lumières
rgb_colorà pleine intensité : une teinte sombre ([128, 0, 128]) revenait[255, 0, 255], doncconfirmed: false. La couleur demandée et la couleur relue sont maintenant comparées à pleine intensité (canal le plus fort ramené à 255), avec la même tolérance.POST /api/admin/home-assistant/controlavecrgbColor(violet sombre envoyé tel quel, confirmé).docs/developpement/architecture.md:home_assistant__controlcite la couleur et le blanc ; la doc Home Assistant précise la comparaison.#34, bouton du satellite
docs/reference/confidentialite.md: en veille, le bouton coupe le micro ; pendant une conversation, il la ferme.features: ["session_end"]duwelcome).#36, fin de parole propre au satellite
satellites.mdetadministration.md: le délai est pris àsatelliteWake, un changement fait pendant une conversation vaut pour la suivante.#38, volume à la voix
main/audio.c,main/board.c) : la sonnerie passe parboard_speaker_writesur le même codec que la voix, dontboard_set_volumerègleesp_codec_dev_set_out_vol. Au volume 0, les sonneries du satellite sont donc muettes aussi (lecture du code, pas mesuré). L'outil le rappelle au volume 0 (« Attention : les réveils et minuteurs de ce satellite seront muets aussi. ») et la doc le dit.Vérifications
npm run check: OK ;prettiersur les fichiers modifiés.home-assistant,home-assistant-api,satellite-volume,satellites: 32/32.CHANGELOG.mdmis à jour.Conflits possibles sur
docs/developpement/architecture.md:134avec #28 et #30, qui touchent la même ligne : je rebaserai selon l'ordre de fusion.