Skip to content

Conversation

@FraBoCH
Copy link
Contributor

@FraBoCH FraBoCH commented Sep 18, 2025

Adding French sentences for HassSetVolumeRelative intent

@FraBoCH FraBoCH requested a review from jlpouffier as a code owner September 18, 2025 20:02
@FraBoCH FraBoCH changed the title [fr] add media player relative volume setting [FR] add media player relative volume setting Sep 22, 2025
Copy link
Member

@jlpouffier jlpouffier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello !

Thx a lot for your PR!

We have common expansion rules for verbs monte and baisse here

Do you mind using them so that the sentences are coherent with other sentences using these verbs?

You can take inspiration from the Absolute Volume setting in case you want to know how to use expansion rules!

@jlpouffier jlpouffier marked this pull request as draft October 23, 2025 07:07
@FraBoCH
Copy link
Contributor Author

FraBoCH commented Oct 25, 2025

Thanks for your review @jlpouffier! I reused the existing expansion as requested

@FraBoCH FraBoCH closed this Oct 26, 2025
@FraBoCH FraBoCH reopened this Oct 26, 2025
@FraBoCH FraBoCH marked this pull request as ready for review October 26, 2025 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants