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

[FR] Enhanced some wordings into french sentences #2707

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

RSickenberg
Copy link

No description provided.

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link
Author

@RSickenberg RSickenberg left a comment

Choose a reason for hiding this comment

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

Missed my replace all by some places.

sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
sentences/fr/binary_sensor_HassGetState.yaml Outdated Show resolved Hide resolved
Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link

@home-assistant home-assistant bot left a comment

Choose a reason for hiding this comment

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

Hi @RSickenberg

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml

Update sentences/fr/binary_sensor_HassGetState.yaml
@RSickenberg RSickenberg force-pushed the french-fix-rsickenberg-patch1 branch from cec82e2 to 2cf91de Compare December 3, 2024 21:26
@RSickenberg RSickenberg marked this pull request as ready for review December 3, 2024 21:27
@RSickenberg
Copy link
Author

All good, sorry for the noise.

Copy link
Member

@piitaya piitaya left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution.

Are you using the battery sensor intents? We are planning to drop most of sensor and binary_sensor (see #2602 for reasons)

@@ -682,7 +682,7 @@ expansion_rules:
# Mets
mets_dirty: "(mais|maître)"
# ferme
ferme_dirty: "faire"
ferme_dirty: "(faire|fêrme)"
Copy link
Member

@piitaya piitaya Dec 4, 2024

Choose a reason for hiding this comment

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

Does a specific STT engine provide that word? It's not a french word, right?

Copy link
Author

@RSickenberg RSickenberg Dec 4, 2024

Choose a reason for hiding this comment

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

In French, it's a "Farm" but without the ^ on the e, I am technically not sure which one could be an error in an STT engine. I can fix it or remove it.

sentences/fr/homeassistant_HassNevermind.yaml Outdated Show resolved Hide resolved
sentences/fr/homeassistant_HassGetCurrentDate.yaml Outdated Show resolved Hide resolved
@RSickenberg RSickenberg requested a review from piitaya December 4, 2024 14:04
@piitaya
Copy link
Member

piitaya commented Dec 4, 2024

PR needs to be rebased as battery intents have been removed.

@RSickenberg RSickenberg requested a review from piitaya December 4, 2024 19:33
piitaya
piitaya previously approved these changes Dec 4, 2024
@piitaya
Copy link
Member

piitaya commented Dec 5, 2024

Ci is failing because of missing test for the new sentence

@piitaya piitaya dismissed stale reviews from home-assistant[bot] and themself December 5, 2024 07:59

cla signed

@piitaya piitaya marked this pull request as draft December 5, 2024 08:00
@jlpouffier
Copy link
Contributor

@RSickenberg What is the state of this PR?
Can you check the failed test, fix it and put it back to open once done?

Thx a lot!

JLo

@RSickenberg
Copy link
Author

RSickenberg commented Dec 14, 2024

@jlpouffier I don't have time to write tests. Feel free to close it or just update the test. :)

Edit: Changed statement about knowledge b.c I can learn. It's just an amount of time.

Best,
Romain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants