Skip to content

Commit 69284d2

Browse files
authored
Merge pull request #134 from Coding-Club-IITG/sanjeebani
Removed delete year option for read only courses
2 parents cadd332 + 7623b7a commit 69284d2

1 file changed

Lines changed: 10 additions & 8 deletions

File tree

  • client/src/screens/browse/components/year-info

client/src/screens/browse/components/year-info/index.jsx

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ const YearInfo = ({
3838
};
3939

4040
if (course?.length > 1) course.sort(sortYear);
41-
41+
4242
const handleAddYear = () => {
4343
setNewYearName("");
4444
setShowConfirm(true);
@@ -137,20 +137,22 @@ const YearInfo = ({
137137
}}
138138
>
139139
{year.name}
140-
{isBR ? (
140+
{isBR && !isReadOnlyCourse ? (
141141
<div
142142
className="delete"
143143
onClick={handleDeleteYear}
144144
title="Delete Year"
145145
></div>
146146
) : null}
147147
</span>
148-
<ConfirmDelDialog
149-
isOpen={showConfirmDel}
150-
type="delete"
151-
onConfirm={handleConfirmDeleteYear}
152-
onCancel={cancelDelete}
153-
/>
148+
{isBR && !isReadOnlyCourse ? (
149+
<ConfirmDelDialog
150+
isOpen={showConfirmDel}
151+
type="delete"
152+
onConfirm={handleConfirmDeleteYear}
153+
onCancel={cancelDelete}
154+
/>
155+
) : null}
154156
</div>
155157
);
156158
})}

0 commit comments

Comments
 (0)