Commit c9f42a01 authored by Peter Hegman's avatar Peter Hegman

Merge branch '354332-inconsistent-wording-remove-member' into 'master'

Rename remove user button in project members

See merge request gitlab-org/gitlab!82109
parents f5269a7d af0a2866
...@@ -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
......
...@@ -22978,9 +22978,6 @@ msgstr "" ...@@ -22978,9 +22978,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