Commit af0a2866 authored by Sascha Eggenberger's avatar Sascha Eggenberger Committed by Peter Hegman

Rename remove user action

Fix inconsistent wording for removing member on project members
overview.
Renamed action "remove user" to "remove member".

Changelog: changed
parent ce215249
...@@ -25,7 +25,8 @@ export default { ...@@ -25,7 +25,8 @@ export default {
}, },
title: { title: {
type: String, type: String,
required: true, required: false,
default: null,
}, },
icon: { icon: {
type: String, type: String,
......
...@@ -61,7 +61,7 @@ export default { ...@@ -61,7 +61,7 @@ export default {
}; };
}, },
removeMemberButtonText() { removeMemberButtonText() {
return this.isInvitedUser ? null : __('Remove user'); return this.isInvitedUser ? null : __('Remove member');
}, },
removeMemberButtonIcon() { removeMemberButtonIcon() {
return this.isInvitedUser ? 'remove' : ''; return this.isInvitedUser ? 'remove' : '';
...@@ -86,7 +86,6 @@ export default { ...@@ -86,7 +86,6 @@ export default {
:icon="removeMemberButtonIcon" :icon="removeMemberButtonIcon"
:button-text="removeMemberButtonText" :button-text="removeMemberButtonText"
:button-category="removeMemberButtonCategory" :button-category="removeMemberButtonCategory"
:title="s__('Member|Remove member')"
/> />
</div> </div>
<div v-else-if="permissions.canOverride && !member.isOverridden" class="gl-px-1"> <div v-else-if="permissions.canOverride && !member.isOverridden" class="gl-px-1">
......
...@@ -163,7 +163,7 @@ To remove a member from a project: ...@@ -163,7 +163,7 @@ To remove a member from a project:
1. On the top bar, select **Menu > Projects** and find your project. 1. On the top bar, select **Menu > Projects** and find your project.
1. On the left sidebar, select **Project information > Members**. 1. On the left sidebar, select **Project information > Members**.
1. Next to the project member you want to remove, select **Remove member** **{remove}**. 1. Next to the project member you want to remove, select **Remove member**.
1. Optional. In the confirmation box, select the 1. Optional. In the confirmation box, select the
**Also unassign this user from related issues and merge requests** checkbox. **Also unassign this user from related issues and merge requests** checkbox.
1. To prevent leaks of sensitive information from private projects, verify the 1. To prevent leaks of sensitive information from private projects, verify the
......
...@@ -22966,9 +22966,6 @@ msgstr "" ...@@ -22966,9 +22966,6 @@ msgstr ""
msgid "Member|Deny access" msgid "Member|Deny access"
msgstr "" msgstr ""
msgid "Member|Remove member"
msgstr ""
msgid "Member|Revoke invite" msgid "Member|Revoke invite"
msgstr "" msgstr ""
......
...@@ -43,12 +43,12 @@ describe('UserActionButtons', () => { ...@@ -43,12 +43,12 @@ describe('UserActionButtons', () => {
memberId: member.id, memberId: member.id,
memberType: 'GroupMember', memberType: 'GroupMember',
message: `Are you sure you want to remove ${member.user.name} from "${member.source.fullName}"?`, message: `Are you sure you want to remove ${member.user.name} from "${member.source.fullName}"?`,
title: 'Remove member', title: null,
isAccessRequest: false, isAccessRequest: false,
isInvite: false, isInvite: false,
icon: '', icon: '',
buttonCategory: 'secondary', buttonCategory: 'secondary',
buttonText: 'Remove user', buttonText: 'Remove member',
userDeletionObstacles: { userDeletionObstacles: {
name: member.user.name, name: member.user.name,
obstacles: parseUserDeletionObstacles(member.user), obstacles: parseUserDeletionObstacles(member.user),
...@@ -135,9 +135,9 @@ describe('UserActionButtons', () => { ...@@ -135,9 +135,9 @@ describe('UserActionButtons', () => {
describe('isInvitedUser', () => { describe('isInvitedUser', () => {
it.each` it.each`
isInvitedUser | icon | buttonText | buttonCategory isInvitedUser | icon | buttonText | buttonCategory
${true} | ${'remove'} | ${null} | ${'primary'} ${true} | ${'remove'} | ${null} | ${'primary'}
${false} | ${''} | ${'Remove user'} | ${'secondary'} ${false} | ${''} | ${'Remove member'} | ${'secondary'}
`( `(
'passes the correct props to remove-member-button when isInvitedUser is $isInvitedUser', 'passes the correct props to remove-member-button when isInvitedUser is $isInvitedUser',
({ isInvitedUser, icon, buttonText, buttonCategory }) => { ({ isInvitedUser, icon, buttonText, buttonCategory }) => {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment