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

make translatable & add german translations #109

Closed
wants to merge 2 commits into from

Conversation

bumbus
Copy link

@bumbus bumbus commented Oct 26, 2023

No description provided.

@@ -36,4 +42,11 @@ public function getURL(): string
{
return $this->Email ? sprintf('mailto:%s', $this->Email) : '';
}

public function fieldLabels($includerelations = true)
Copy link
Member

@GuySartorelli GuySartorelli Oct 30, 2023

Choose a reason for hiding this comment

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

This isn't necessary - there's a fallback to a dB namespaced translatable key which you'll surface in the yml file if you run the i18n text collector.

Same with all other field labels you're adding

Copy link
Author

Choose a reason for hiding this comment

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

Hmm am I missing something, if I comment out fieldLabels function the translations are not applied.

Bildschirmfoto 2023-10-30 um 16 28 16

Copy link
Member

Choose a reason for hiding this comment

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

Did you run the i18n text collector over this module, and use the new translation keys it generates for your german translations?

client/lang/de.js Outdated Show resolved Hide resolved
@GuySartorelli
Copy link
Member

Thanks for your interest in this module and for the contribution.
Just an FYI, this module is still undergoing active development and we'll likely make this fully translatable as part of that development. As a result it's likely that we'll do that work separately and that work will supercede this pull request.

@GuySartorelli
Copy link
Member

Closing in favour of #119

Once v4 is closer to release we'll push the strings to transifex and you can add your German translations there.

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