Skip to content

Commit 3aa41c9

Browse files
committed
Implement review feedback
1 parent d4d3401 commit 3aa41c9

File tree

2 files changed

+6
-9
lines changed

2 files changed

+6
-9
lines changed

src/components/Members/EditMembers.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@ export const ACCOUNT_TYPES: RadioButtonsSelectOption[] = [
2525

2626
export type AccountType = 'User' | 'ServiceAccount';
2727

28-
const removeProjectPrefix = (name?: string) => (name?.startsWith('project-') ? name.slice('project-'.length) : name);
28+
const PROJECT_PREFIX = 'project-';
29+
const removeProjectPrefix = (name?: string) =>
30+
name?.startsWith(PROJECT_PREFIX) ? name.slice(PROJECT_PREFIX.length) : name;
2931

3032
export const EditMembers: FC<EditMembersProps> = ({
3133
members,

src/components/Members/ImportMembersDialog.tsx

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -212,19 +212,14 @@ export const ImportMembersDialog: FC<ImportMembersDialogProps> = ({
212212
};
213213

214214
function getAddMembersButtonText(selectedMembersCount: number, t: TFunction) {
215-
let addButtonText = '';
216215
switch (selectedMembersCount) {
217216
case 0:
218-
addButtonText = t('ImportMembersDialog.addMembersButton0');
219-
break;
217+
return t('ImportMembersDialog.addMembersButton0');
220218
case 1:
221-
addButtonText = t('ImportMembersDialog.addMembersButton1');
222-
break;
219+
return t('ImportMembersDialog.addMembersButton1');
223220
default:
224-
addButtonText = t('ImportMembersDialog.addMembersButtonN', { count: selectedMembersCount });
225-
break;
221+
return t('ImportMembersDialog.addMembersButtonN', { count: selectedMembersCount });
226222
}
227-
return addButtonText;
228223
}
229224

230225
interface SpecMembers {

0 commit comments

Comments
 (0)