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

Update German translations, add translation of current state to settings page #32

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

eengnr
Copy link
Contributor

@eengnr eengnr commented Feb 23, 2024

Hi! Thanks a lot for this great project!
I'm still waiting for some hardware parts, but playing around with the software alone was fun already 🙂

And everything is so well documented, I really like it!

I optimized the German translation and added a translation for the current state to the settings page. I hope this can be added!

Comment on lines +177 to +186
getReadableActivityState(activityState: string): string {
switch (activityState) {
case 'Winding':
return this.translateService.instant('SETTINGS.WINDING');
case 'Stopped':
return this.translateService.instant('SETTINGS.STOPPED');
default:
return this.translateService.instant('SETTINGS.STOPPED');
}
}
Copy link
Owner

Choose a reason for hiding this comment

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

great! 👍

@mwood77
Copy link
Owner

mwood77 commented Feb 23, 2024

Thanks for the contribution and navigating through the front end codebase - it's a bit of a mess, and it's on my list of things to fix 😅

I'll merge your work in and rebuild the front end's filesystem on main.

@mwood77 mwood77 merged commit f912020 into mwood77:main Feb 23, 2024
3 checks passed
@eengnr eengnr deleted the german-translation branch February 24, 2024 15:40
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.

2 participants