phetsims / phetsims/rosetta

Short-term string storage can cause problems when multiple translators are working on the same sim

Open
#447 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
3
Forks
3
PR merge metrics
No merged PRs in 30d

Description

In https://github.com/phetsims/rosetta/issues/446, it was reported that strings for a fully translated sim were not being shown in Rosetta's API for a specific translator. The explanation turned out to be that previously saved strings were being shown instead of the currently published strings. This reveals what I feel is a weakness in the current implementation. Rosetta allows users to save strings temporarily until a translation is ready to be fully submitted. However, if there are multiple translators working on the same sim, the strings saved in short-term storage can get out of sync with those that have been submitted for publication. Now that more translators are using Rosetta, this situation is likely to come up more often, and we should probably improve its behavior in this regard.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading issue #446 and tracing how Rosetta handles short-term saved strings versus currently published strings. Inspect the relevant storage and API entry points to understand how concurrent translators can see inconsistent data. Done means the behavior is defined and temporary strings no longer incorrectly override or conflict with published translations.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
localization
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.