Skip to content

Commit 68a5b00

Browse files
authored
Merge pull request #1418 from openedx/fix--course-assignment-group-bug
fix: assign button grey out
2 parents 1010d7d + 4419d25 commit 68a5b00

File tree

2 files changed

+500
-315
lines changed

2 files changed

+500
-315
lines changed

src/components/learner-credit-management/assignment-modal/AssignmentModalContent.jsx

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -59,28 +59,26 @@ const AssignmentModalContent = ({
5959
setDropdownToggleLabel,
6060
setGroupMemberEmails,
6161
});
62-
const handleEmailAddressInputChange = (e) => {
63-
const inputValue = e.target.value;
64-
setEmailAddressesInputValue(inputValue);
65-
};
62+
6663
const handleEmailAddressesChanged = useCallback((value) => {
6764
if (!value) {
6865
setLearnerEmails([]);
69-
onEmailAddressesChange([]);
7066
return;
7167
}
7268
const emails = value.split('\n').filter((email) => email.trim().length > 0);
7369
setLearnerEmails(emails);
74-
}, [onEmailAddressesChange]);
70+
}, []);
7571

7672
const debouncedHandleEmailAddressesChanged = useMemo(
7773
() => debounce(handleEmailAddressesChanged, EMAIL_ADDRESSES_INPUT_VALUE_DEBOUNCE_DELAY),
7874
[handleEmailAddressesChanged],
7975
);
8076

81-
useEffect(() => {
82-
debouncedHandleEmailAddressesChanged(emailAddressesInputValue);
83-
}, [emailAddressesInputValue, debouncedHandleEmailAddressesChanged]);
77+
const handleEmailAddressInputChange = (e) => {
78+
const inputValue = e.target.value;
79+
debouncedHandleEmailAddressesChanged(inputValue);
80+
setEmailAddressesInputValue(inputValue);
81+
};
8482

8583
useEffect(() => {
8684
handleGroupsChanged(checkedGroups);

0 commit comments

Comments
 (0)