-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: Rename to teams #1512
feat: Rename to teams #1512
Conversation
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.
See small remarks, otherwise looks good
@@ -2,7 +2,7 @@ | |||
<info xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | |||
xsi:noNamespaceSchemaLocation="https://apps.nextcloud.com/schema/apps/info.xsd"> | |||
<id>circles</id> | |||
<name>Circles</name> | |||
<name>Teams</name> | |||
<summary>Bring cloud-users closer together.</summary> |
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.
This could use a better summary, maybe from @jospoortvliet?
appinfo/info.xml
Outdated
@@ -2,7 +2,7 @@ | |||
<info xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | |||
xsi:noNamespaceSchemaLocation="https://apps.nextcloud.com/schema/apps/info.xsd"> | |||
<id>circles</id> | |||
<name>Circles</name> | |||
<name>Teams</name> | |||
<summary>Bring cloud-users closer together.</summary> | |||
<description><![CDATA[Circles allows people to create their own groups of people/colleagues/friends. |
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.
Name needs to be changed in the description too (not only this line), right?
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.
Adjusted, also in the README, we can update the description once the marketing message is clearer as well I'd say.
Signed-off-by: Julius Härtl <jus@bitgrid.net>
cafd53b
to
c5fd973
Compare
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.
Looks good to me except some comments.
lib/Activity/SettingAsModerator.php
Outdated
@@ -51,7 +51,7 @@ public function getIdentifier() { | |||
* @since 11.0.0 | |||
*/ | |||
public function getName() { | |||
return $this->l10n->t('Any important event in a <strong>Circle</strong> you are moderating'); | |||
return $this->l10n->t('Any important event in a <strong>Team</strong> you are moderating'); |
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.
Since you use lowercase 'team' elsewhere, maybe also adjust it here?
Signed-off-by: Julius Härtl <jus@bitgrid.net>
c5fd973
to
32a6efa
Compare
@mejo- Thanks, adjusted |
Mostly replaced as App name, icon and in translation strings