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

i18n: Need context, sentence needs improvement #6408

Closed
rakekniven opened this issue Feb 4, 2024 · 4 comments · Fixed by #6703
Closed

i18n: Need context, sentence needs improvement #6408

rakekniven opened this issue Feb 4, 2024 · 4 comments · Fixed by #6703

Comments

@rakekniven
Copy link
Member

emit finished(403, tr("Could not add or remove a folder user %1, for folder %2").arg(_folderUserId).arg(_path));

folder user -> user folder?

Please improve grammar

@mgallien
Copy link
Collaborator

mgallien commented Feb 5, 2024

@rakekniven I will see internally
The user here is an user having access to the folder that is a shared folder with end-to-end encryption.
Does this may work for you:
"Could not add or remove user %1 to access folder %2"

@rakekniven
Copy link
Member Author

Does this may work for you: "Could not add or remove user %1 to access folder %2"

Much better.

@rakekniven
Copy link
Member Author

@mgallien Will you change it?

@rakekniven
Copy link
Member Author

Ping

rakekniven added a commit that referenced this issue Apr 25, 2024
Fixes #6408 

Reported at Transifex.

Signed-off-by: rakekniven <[email protected]>
claucambra pushed a commit that referenced this issue Apr 28, 2024
Fixes #6408 

Reported at Transifex.

Signed-off-by: rakekniven <[email protected]>
backportbot bot pushed a commit that referenced this issue Apr 28, 2024
Fixes #6408 

Reported at Transifex.

Signed-off-by: rakekniven <[email protected]>
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 a pull request may close this issue.

2 participants