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

Added 'Speaker' and 'Battery Service' component #500

Merged
merged 2 commits into from
Jan 23, 2020

Conversation

LaborEtArs
Copy link

Was a long way to go... but now:
Added 'Speaker' and 'Battery Service' component!
Both components are mostly 'stolen' from 'Lightbulb' and 'Thermostat'.
The translations are from 'google', so there is definitely potential for optimizations ;-)

@oznu
Copy link
Member

oznu commented Jan 22, 2020

Thanks @LaborEtArs, this is an amazing effort. I'll test this out soon.

Copy link
Contributor

@Caribsky Caribsky left a comment

Choose a reason for hiding this comment

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

small fix ru.json

@@ -18,6 +18,10 @@
"accessories.control.label_on": "Вкл",
"accessories.control.label_open": "Открыто",
"accessories.control.label_opening": "Открывается",
"accessories.control.label_batteryservice_notcharging": "Не заряжается",
"accessories.control.label_batteryservice_charging": "Зарядка",
"accessories.control.label_batteryservice_notchargeable": "Не оплачивается",
Copy link
Contributor

@Caribsky Caribsky Jan 23, 2020

Choose a reason for hiding this comment

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

The correct translation in case when charging is not supported:

Suggested change
"accessories.control.label_batteryservice_notchargeable": "Не оплачивается",
"accessories.control.label_batteryservice_notchargeable": "Без зарядки",

"accessories.control.label_batteryservice_notcharging": "Не заряжается",
"accessories.control.label_batteryservice_charging": "Зарядка",
"accessories.control.label_batteryservice_notchargeable": "Не оплачивается",
"accessories.control.label_speaker_volume": "объем",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"accessories.control.label_speaker_volume": "объем",
"accessories.control.label_speaker_volume": "Громкость",

@oznu oznu merged commit c851853 into homebridge:master Jan 23, 2020
@oznu
Copy link
Member

oznu commented Jan 23, 2020

Ah I can't add @Caribsky changes after merging. Let me try something, I might break this PR..

@oznu
Copy link
Member

oznu commented Jan 23, 2020

I applied them manually.

@oznu
Copy link
Member

oznu commented Jan 23, 2020

Thanks @LaborEtArs and @Caribsky for your contribution.

@oznu
Copy link
Member

oznu commented Jan 23, 2020

This is currently in @test. Since I just did a release yesterday I'm going to hold off publishing this to everyone as I don't want users to get update fatigue and there are a few unrelated bugs that need resolving.

You can install the test version using:

sudo npm install -g --unsafe-perm homebridge-config-ui-x@test

@Caribsky
Copy link
Contributor

I have tested the new battery service component only. Well done! Thanks guys

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

Successfully merging this pull request may close these issues.

3 participants