Strings fetched from the server are not translated #172

Open
opened 2023-01-02 03:26:19 +02:00 by def · 7 comments
Collaborator

image

![image](/attachments/222564b6-4762-4eb9-8552-2861ca8fae6b)
167 KiB
def added the
Bug
Priority
Medium
labels 2023-01-02 03:26:19 +02:00
NaiJi was assigned by def 2023-01-02 03:27:02 +02:00

These are fetched from the server, i18n has to be implemented at the API level.

These are fetched from the server, i18n has to be implemented at the API level.
NaiJi was unassigned by inex 2023-01-04 11:26:36 +02:00
inex removed the
Bug
label 2023-01-04 11:26:40 +02:00
inex changed title from Translation errors to Strings fetched from the server are not translated 2023-01-04 11:26:57 +02:00
Collaborator

Related issue: 205.
Is partially pending on: PR 243.

Related issue: [205](https://git.selfprivacy.org/SelfPrivacy/selfprivacy.org.app/issues/205). Is partially pending on: [PR 243](https://git.selfprivacy.org/SelfPrivacy/selfprivacy.org.app/pulls/243).
Collaborator

Why exactly are they fetched from the server at all?
If these are static explainers, they should be a part of app interface with all the translations I think.

Why exactly are they fetched from the server at all? If these are *static* explainers, they should be a part of app interface with all the translations I think.

They are not static from the app POV. There is no point of updating the app every time we add a new service or change something in API.

They are not static from the app POV. There is no point of updating the app every time we add a new service or change something in API.
Collaborator

Funny enough that the only translated part on the OP screenshot is TODO-covered with this exact issue

/// TODO Turn loginInfo into dynamic data, not static!
  String get loginInfo {
    switch (id) {
      case 'mailserver':
        return 'mail.login_info'.tr();
      case 'bitwarden':
        return 'password_manager.login_info'.tr();
      . . .
    }
    return '';
  }
Funny enough that the only translated part on the OP screenshot is TODO-covered with this exact issue ```dart /// TODO Turn loginInfo into dynamic data, not static! String get loginInfo { switch (id) { case 'mailserver': return 'mail.login_info'.tr(); case 'bitwarden': return 'password_manager.login_info'.tr(); . . . } return ''; } ```
Poster
Collaborator

what the status?

what the status?
def added
Bug
Severity
Medium
and removed
Priority
Medium
labels 2023-08-30 16:46:48 +03:00

Implemented on the client side, implementation pending on API side.

Implemented on the client side, implementation pending on API side.
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: SelfPrivacy/selfprivacy.org.app#172
There is no content yet.