Fix notebook rename.

server/app/services/notebook.py

@@ -243,6 +243,8 @@ def rename_notebook_by_path(notebook_path: str = None, new_notebook_name: str =
     parent_path = '/'.join(notebook_path.split('/')[:-1])
     new_path = f"{parent_path}/{new_notebook_name}"
 
+    logger.info(f"Renaming notebook with path: {notebook_path} to {new_path}")
+
     response = requests.patch(
       path,
       json={"path": new_path}

webapp/src/components/notebook/Notebook.js

@@ -107,7 +107,7 @@ function Notebook({
                 path: 'work/' + currentName,
             }
         });
-        NotebookModel.renameNotebook(baseUrl, notebook.path, currentName).then((data) => {
+        NotebookModel.renameNotebook(notebook.path, currentName).then((data) => {
             console.log('Notebook name saved:', data);
         }).catch((error) => {
             console.error('Failed to save notebook name:', error);

webapp/src/components/sidebar/workspace/item/MoreButton.js

@@ -3,6 +3,7 @@ import { CgMoreVerticalAlt } from "react-icons/cg";
 import { Menu, MenuItem } from '@mui/material';
 import config from '../../../../config';
 import DirectoryModel from '../../../../models/DirectoryModel';
+import NotebookModel from '../../../../models/NotebookModel';
 import DeleteDialog from './DeleteDialog';
 import RenameDialog from './RenameDialog';
 
@@ -38,7 +39,11 @@ const MoreButton = ({
 
   const handleRename = async (file, newName) => {
     try {
-      await DirectoryModel.renameItem(currentPath + '/'  + file.name, currentPath + '/' + newName);
+      if (file.type === 'notebook') {
+        await NotebookModel.renameNotebook(currentPath + '/' + file.name, newName);
+      } else {
+        await DirectoryModel.renameDirectory(currentPath + '/'  + file.name, currentPath + '/' + newName);
+      }
     } catch (error) {
       console.error('Failed to rename item:', error);
     }
@@ -98,7 +103,6 @@ const MoreButton = ({
             Rename
         </MenuItem>
         <RenameDialog 
-          baseUrl={baseUrl}
           file={file}
           renameDialogOpen={renameDialogOpen}
           setRenameDialogOpen={setRenameDialogOpen}

webapp/src/components/sidebar/workspace/item/RenameDialog.js

@@ -3,15 +3,14 @@ import { useState } from 'react';
 import NotebookModel from '../../../../models/NotebookModel';
 
 const RenameDialog = ({
-  baseUrl, 
   file, 
   renameDialogOpen, 
   setRenameDialogOpen,
   handleMoreClose, 
   handleRename
 }) => {
 
-  const [newName, setNewName] = useState(file.name);
+  const [newName, setNewName] = useState(NotebookModel.getNameWithoutExtension(file.name));
 
   const handleInputChange = (event) => {
     setNewName(event.target.value);
@@ -68,7 +67,7 @@ const RenameDialog = ({
         </Button>
         <Button 
           style={{ color: 'lightgrey' }}
-          onClick={(event) => {
+          onClick={() => {
             setRenameDialogOpen(false);
             handleMoreClose();
             file.type === 'directory' ? 

webapp/src/models/DirectoryModel.js

@@ -71,7 +71,7 @@ class DirectoryModel {
     }
   }; 
 
-  static async renameItem(oldPath='', newPath='') {
+  static async renameDirectory(oldPath='', newPath='') {
     console.log("Renaming item at path:", oldPath, "to", newPath);
     const response = await fetch("http://localhost:5002/directory/" + oldPath, {
         method: 'PATCH',

webapp/src/models/NotebookModel.js

@@ -157,7 +157,7 @@ class NotebookModel {
     });
 
     if (!response.ok) {
-        throw new Error('Failed to create notebook');
+        throw new Error('Failed to delete notebook');
     } else {
         const data = await response.json();
         return data;
@@ -194,8 +194,8 @@ class NotebookModel {
     return data;
   };
 
-  static async renameNotebook(basePath = '', path = '', newName = '') {
-    console.log("Renaming notebook at path:", basePath + '/' + path, "to:", newName);
+  static async renameNotebook(path = '', newName = '') {
+    console.log("Renaming notebook at path:", path, "to:", newName);
 
     const response = await fetch("http://localhost:5002/notebook/" + path, {
         method: 'PATCH',