Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

Commit d91d434

Browse files
committed
fix: add members case-insensitive way
ref issue #14
1 parent 4cb9c59 commit d91d434

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

src/routes/TeamAccess/components/AddModal/index.jsx

+4-4
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ import { getMemberSuggestions } from "services/teams";
1414
/**
1515
* Fetches suggestions based on input in select box
1616
* @param {string} inputVal Input from select
17-
*
18-
* @returns {Promise<Array>} A promise that resolves to list of suggested users
17+
*
18+
* @returns {Promise<Array>} A promise that resolves to list of suggested users
1919
*/
2020
const loadSuggestions = inputVal => {
2121
return getMemberSuggestions(inputVal)
@@ -29,7 +29,7 @@ const loadSuggestions = inputVal => {
2929
.catch(() => {
3030
console.warn("could not get suggestions");
3131
return [];
32-
})
32+
})
3333
}
3434

3535
/**
@@ -104,7 +104,7 @@ const AddModal = ({ open, onClose, teamId, validateAdds, showSuggestions }) => {
104104
selectedMembers.forEach((member) => {
105105
const val = member.label;
106106
if (member.isEmail) {
107-
emails.push(val.toLowerCase());
107+
emails.push(val);
108108
} else {
109109
handles.push(val);
110110
}

src/routes/TeamAccess/components/AddModalContainer/index.jsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@ import { PERMISSIONS } from "constants/permissions";
1818
*/
1919
const checkForMatches = (newMember, memberList) => {
2020
const label = newMember.label;
21+
const lowered = label.toLowerCase();
2122

2223
if (newMember.isEmail) {
23-
const lowered = label.toLowerCase();
2424
return memberList.find((member) => {
25-
return member.email === lowered;
25+
return member.email.toLowerCase() === lowered;
2626
});
2727
}
28-
return memberList.find((member) => member.handle === label);
28+
return memberList.find((member) => member.handle.toLowerCase() === lowered);
2929
};
3030

3131
const AddModalContainer = ({

0 commit comments

Comments
 (0)