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

[Bug]: Manual install title is wrong in non-English #2394

Open
sbruens opened this issue Feb 15, 2025 · 2 comments
Open

[Bug]: Manual install title is wrong in non-English #2394

sbruens opened this issue Feb 15, 2025 · 2 comments
Labels
bug Something is broken, we are tracking it good first issue Issues that are good for a beginner or new contributor to tackle manager

Comments

@sbruens
Copy link
Contributor

sbruens commented Feb 15, 2025

Application

Outline Manager

Describe the bug

We're doing string replacement in the manual installation title, but are not always providing a value, which is bad. For non-Cloud Provider setup we shouldn't leverage the same translation message.

Steps to reproduce

  1. Open the Manager
  2. Change to non-English language
  3. Set up manual server

What did you expect to happen?

No response

What actually happened?

Panel title is cut off because the Cloud Provider is an empty string, which doesn't make sense in the message in non-English languages.

Outline Version

1.16.0

What operation system are you using?

macOS

Operating System Version

No response

Screenshots and Videos

Image
@sbruens sbruens added bug Something is broken, we are tracking it manager good first issue Issues that are good for a beginner or new contributor to tackle labels Feb 15, 2025
@sbruens sbruens changed the title [Bug]: Manual install title does not translate [Bug]: Manual install title is wrong in non-English Feb 15, 2025
@Arnesh-pal
Copy link

I'd be happy to help!
Could you share more details about the issue?

@sbruens
Copy link
Contributor Author

sbruens commented Feb 16, 2025

@Arnesh-pal thanks for offering to help! We appreciate contributions.

In the Manager, click the "Advanced - Set up Outline anywhere" panel. This page has the title "These steps will help you install Outline on a Linux server." but in other languages this is incorrect.

Image

This is because we try and re-use the message for all installation panels, using a parameter cloudProvider in the message. See the message:

"manual-server-description": "These steps will help you install Outline on a {cloudProvider} Linux server.",
and where we use it:
<span slot="step-description"
>[[localize('manual-server-description', 'cloudProvider',
cloudProviderName)]]</span
>

For all other panels (GCP, DigitalOcean, etc.) this makes sense, but for a manual install we use an empty string for cloudProvider. This genitive case happens to work in English, but it doesn't work in other languages. We should use a dedicated message for this that doesn't expect any cloudProvider message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken, we are tracking it good first issue Issues that are good for a beginner or new contributor to tackle manager
Projects
None yet
Development

No branches or pull requests

2 participants