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

Rename users to account or person #34178

Merged
merged 5 commits into from
Feb 13, 2024
Merged

Conversation

PVince81
Copy link
Member

@PVince81 PVince81 commented Sep 21, 2022

Replace translated texts from "user" to either "account" or "people" depending on context

Resolves #32720

@PVince81 PVince81 added the 3. to review Waiting for reviews label Sep 21, 2022
@PVince81 PVince81 added this to the Nextcloud 25 milestone Sep 21, 2022
@PVince81 PVince81 self-assigned this Sep 21, 2022
@PVince81 PVince81 force-pushed the replace-user-to-account-or-person branch from 8d4de1e to 79d887b Compare September 21, 2022 16:36
@PVince81
Copy link
Member Author

I've fixed the unit tests locally, I'll push after I see the acceptance test results... some rely on texts/labels sometimes

@PVince81
Copy link
Member Author

this is getting complicated because the users page is using some core strings so I'd need to replace some of them

and in #34179 replace the rest.

so, let's do this: I'll merge #34179 into this PR and fix the tests at once

@PVince81
Copy link
Member Author

I've pushed even more renames here

@PVince81
Copy link
Member Author

/compile amend /

@PVince81 PVince81 force-pushed the replace-user-to-account-or-person branch 2 times, most recently from 42e100f to 83fc72a Compare September 21, 2022 21:46
@PVince81 PVince81 added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Sep 22, 2022
@PVince81
Copy link
Member Author

as it's a bit late considering the string freeze and also the fact that other apps need adjusting as well, let's not merge this for 25.0.0 but 25.0.1 could do

this would give us time also to properly review all the strings that were replaced and see how they look like in context

@blizzz blizzz mentioned this pull request Sep 22, 2022
2 tasks
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@jancborchardt
Copy link
Member

how to convert:

  • "user status"
  • "User documentation"

Where is "user status" used in these cases? Cause just "Status" might be enough? "the status feature", "set your status", "people can set their status" etc.

For "user documentation" → "Usage documentation". German equivalent would be "Benutzerhandbuch" → "Benutzungshandbuch".

@PVince81
Copy link
Member Author

@jancborchardt I rechecked and it seems the only place where "User status" is visible is for admins as it's the name of the app that can be disabled. And it appears in few debug messages, but that's all. Should we leave it as is ?

@PVince81 PVince81 force-pushed the replace-user-to-account-or-person branch from 83fc72a to 8b50e5e Compare October 19, 2022 15:39
@PVince81
Copy link
Member Author

Rebased and renamed "User documentation" to "Usage documentation"

@PVince81 PVince81 added 3. to review Waiting for reviews enhancement and removed 2. developing Work in progress labels Oct 19, 2022
@@ -163,7 +163,7 @@ public function parseShortVersion(IEvent $event, IEvent $previousEvent = null) {

if (!isset($parsedParameters['user'])) {
// External user via public link share
$subject = str_replace('{user}', $this->activityLang->t('"remote user"'), $subject);
$subject = str_replace('{user}', $this->activityLang->t('"remote person"'), $subject);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not 'remote account' ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jancborchardt for feedback

apps/files_sharing/appinfo/info.xml Outdated Show resolved Hide resolved
@AndyScherzinger

This comment was marked as resolved.

@skjnldsv skjnldsv self-assigned this Jan 24, 2024
@skjnldsv skjnldsv force-pushed the replace-user-to-account-or-person branch from 2858f49 to 693e2cf Compare January 24, 2024 10:48
@skjnldsv skjnldsv force-pushed the replace-user-to-account-or-person branch 4 times, most recently from aa2c934 to 42d99d1 Compare February 13, 2024 16:44
@skjnldsv skjnldsv force-pushed the replace-user-to-account-or-person branch from 42d99d1 to a00b92f Compare February 13, 2024 19:45
@skjnldsv skjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Feb 13, 2024
PVince81 and others added 5 commits February 13, 2024 21:06
Replace translated text in most locations

Signed-off-by: Vincent Petry <[email protected]>
Signed-off-by: John Molakvoæ <[email protected]>
Signed-off-by: John Molakvoæ <[email protected]>
Signed-off-by: John Molakvoæ <[email protected]>
@skjnldsv skjnldsv force-pushed the replace-user-to-account-or-person branch from a00b92f to 524ed97 Compare February 13, 2024 20:08
@skjnldsv skjnldsv merged commit 6f1d9eb into master Feb 13, 2024
155 checks passed
@skjnldsv skjnldsv deleted the replace-user-to-account-or-person branch February 13, 2024 20:28
@blizzz blizzz mentioned this pull request Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Gender-neutral language
10 participants