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 SetFqdnModal labels for hostname and domain creation #632

Merged
merged 1 commit into from
May 24, 2024
Merged

Conversation

stephdl
Copy link
Contributor

@stephdl stephdl commented May 21, 2024

This pull request updates the labels for the hostname and domain fields in the SetFqdnModal component. The labels were changed to "init.hostname_create" and "init.domain_create" respectively. This change improves the clarity and accuracy of the labels in the user interface.

NethServer/dev#6927

@stephdl stephdl self-assigned this May 21, 2024
Copy link
Member

@DavidePrincipi DavidePrincipi left a comment

Choose a reason for hiding this comment

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

I understand the clarity point, but I'd expect some change is also needed in the .json files with l10n strings, right?

@stephdl
Copy link
Contributor Author

stephdl commented May 24, 2024

either we add a new string in the json file init.domain and init.hostname or we decide to reuse the init.domain_create and init.hostname_create like I did

other way we could decide that this panel cannot use the init.* in the json file and it deserves its own translation

@DavidePrincipi DavidePrincipi requested review from andre8244 and removed request for DavidePrincipi May 24, 2024 14:31
@andre8244
Copy link
Contributor

Found the commit that caused the translation error: 6cb07ee

@stephdl
Copy link
Contributor Author

stephdl commented May 24, 2024

what do you think it could be the better approach @andre8244 ?

@andre8244
Copy link
Contributor

what do you think it could be the better approach @andre8244 ?

This PR is ok, already approved :)

@stephdl stephdl merged commit 8f5f22c into main May 24, 2024
2 checks passed
@stephdl stephdl deleted the sdl-6927 branch May 24, 2024 15:59
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