-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(state): use needs-checking/needs-rewriting states #16972
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
base: main
Are you sure you want to change the base?
feat(state): use needs-checking/needs-rewriting states #16972
Conversation
| ): | ||
| unit.state = STATE_FUZZY | ||
| # TODO: needs-checking or needs-rewriting as this is source? | ||
| unit.state = STATE_NEEDS_CHECKING |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs checking seems appropriate here.
| if scope == "report": | ||
| if component.has_template(): | ||
| if unit_scope.translated and not unit_scope.readonly: | ||
| # TODO: needs-checking for target and needs-rewriting for source? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs checking for source, the target should get needs rewriting once the source is edited.
|
The ultimate goal here is to be able to distinguish what triggered the change from the state and what is the desired action. On the other side, we would rather not expose three similarly named states for selection to the translator. What about using one state selection with another input for detailed state, like outlined in #2240 (comment)? In the end, what we're trying to represent is a subset of what
That makes me think whether the names we currently have are clear enough to understand... The states, IMHO, should be written to file when possible. PS: Beware that any UI changes here will most likely run into conflicts with #16882. |

TBA