Merge pull request #1479 from NYPL/MLN-1304

added currentuser condition
NYPL · Dec 30, 2024 · 3628b50 · 3628b50
2 parents 27bc9f7 + d375e34
commit 3628b50
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/javascript/components/TeacherSetDetails/TeacherSetDetails.jsx b/app/javascript/components/TeacherSetDetails/TeacherSetDetails.jsx
@@ -534,7 +534,7 @@ export default function TeacherSetDetails(props) {
   };
 
   const inactiveSchoolMessage = () => {
-    if (currentUserStatus !== undefined && isSchoolActive !== "" && isSchoolActive == false) {
+    if (currentUserStatus && isSchoolActive === false) {
       return (<Banner content={<>
         Your school is inactive, so your account is restricted. Please contact [email protected].
       </>} type="warning" />)

diff --git a/app/models/teacher_set.rb b/app/models/teacher_set.rb
@@ -133,6 +133,7 @@ def update_teacher_set_availability_in_elastic_search
     body = {
      :availability => self.availability,
      :available_copies => self.available_copies,
+     :total_copies => self.total_copies,
     }
     ElasticSearch.new.update_document_by_id(self.id, body)
   end