Skip to content

Conversation

chuk-a
Copy link
Contributor

@chuk-a chuk-a commented Jul 3, 2025

No description provided.

Copy link
Contributor

@Lesekater Lesekater left a comment

Choose a reason for hiding this comment

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

Hi @chuk-a
thanks for you contribution
Since this pr adds a new intent to the language, we also need new test cases (see https://github.com/OHF-Voice/intents/blob/main/tests/en/weather_HassGetWeather.yaml as reference), corresponding fixtures and responses.
thx 👍

@andreasbrett andreasbrett changed the title Create weather_HassGetWeather.yaml [MN] Create weather_HassGetWeather.yaml Jul 9, 2025
chuk-a and others added 2 commits October 5, 2025 10:05
@chuk-a
Copy link
Contributor Author

chuk-a commented Oct 5, 2025

How can I open a test case?

@Lesekater
Copy link
Contributor

Lesekater commented Oct 5, 2025

How can I open a test case?

Just replicate the files in en - so create tests at https://github.com/OHF-Voice/intents/blob/main/tests/mn/weather_HassGetWeather.yaml and fixtures in https://github.com/OHF-Voice/intents/blob/main/tests/mn/_fixtures.yaml and responses in https://github.com/OHF-Voice/intents/blob/main/responses/mn/HassGetWeather.yaml.

Also look at the files from en I linked in #3241 (review)

You can find a lot of documentation about this here and here

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