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

[DE] Add HassBroadcast #2882

Merged
merged 11 commits into from
Jan 17, 2025
Merged

Conversation

andreasbrett
Copy link
Contributor

@andreasbrett andreasbrett commented Jan 17, 2025

DE version of #2877

@mib1185 @easterapps this needs fine tuning. suggestions very welcome. "verkünde, dass das abendessen fertig ist" would need to require a transformed response "das abendessen ist fertig" to be perfectly correct. unfortunately it's not as easy in DE as it is in EN. I could not find phrases that are as smooth as in EN.

Copy link
Contributor

@mib1185 mib1185 left a comment

Choose a reason for hiding this comment

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

what do you think about these ones?

responses/de/HassBroadcast.yaml Outdated Show resolved Hide resolved
sentences/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
tests/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
@mib1185
Copy link
Contributor

mib1185 commented Jan 17, 2025

Sentences should only contain words and matching syntax. They should not contain punctuation.

you're right, we should not add the punctuation 😬

@andreasbrett andreasbrett marked this pull request as ready for review January 17, 2025 22:31
@andreasbrett andreasbrett requested a review from mib1185 January 17, 2025 22:31
@andreasbrett andreasbrett enabled auto-merge (squash) January 17, 2025 22:32
sentences/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
tests/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
@home-assistant home-assistant bot marked this pull request as draft January 17, 2025 22:39
auto-merge was automatically disabled January 17, 2025 22:39

Pull request was converted to draft

@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@andreasbrett andreasbrett marked this pull request as ready for review January 17, 2025 22:45
@home-assistant home-assistant bot requested a review from mib1185 January 17, 2025 22:45
sentences/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
tests/de/assist_satellite_HassBroadcast.yaml Outdated Show resolved Hide resolved
@home-assistant home-assistant bot marked this pull request as draft January 17, 2025 22:47
@mib1185 mib1185 marked this pull request as ready for review January 17, 2025 22:49
@home-assistant home-assistant bot requested a review from mib1185 January 17, 2025 22:49
@andreasbrett andreasbrett merged commit 0c77f93 into home-assistant:main Jan 17, 2025
2 checks passed
@andreasbrett andreasbrett deleted the HassBroadcast branch January 22, 2025 20:30
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.

2 participants