Skip to content

Commit ddff544

Browse files
committed
refactor: Remove lowercase logic from notebook names checks
Signed-off-by: Tomas Kislan <[email protected]>
1 parent 673a070 commit ddff544

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/notebooks/deepnote/deepnoteExplorerView.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -131,13 +131,11 @@ export class DeepnoteExplorerView {
131131
*/
132132
private generateSuggestedNotebookName(projectData: DeepnoteFile): string {
133133
const notebookCount = projectData.project.notebooks?.length || 0;
134-
const existingNames = new Set(
135-
projectData.project.notebooks?.map((nb: DeepnoteNotebook) => nb.name.toLowerCase()) || []
136-
);
134+
const existingNames = new Set(projectData.project.notebooks?.map((nb: DeepnoteNotebook) => nb.name) || []);
137135

138136
let nextNumber = notebookCount + 1;
139137
let suggestedName = `Notebook ${nextNumber}`;
140-
while (existingNames.has(suggestedName.toLowerCase())) {
138+
while (existingNames.has(suggestedName)) {
141139
nextNumber++;
142140
suggestedName = `Notebook ${nextNumber}`;
143141
}
@@ -261,7 +259,7 @@ export class DeepnoteExplorerView {
261259
const suggestedName = this.generateSuggestedNotebookName(projectData);
262260
const notebookName = await this.promptForNotebookName(
263261
suggestedName,
264-
new Set(projectData.project.notebooks?.map((nb: DeepnoteNotebook) => nb.name.toLowerCase()) ?? [])
262+
new Set(projectData.project.notebooks?.map((nb: DeepnoteNotebook) => nb.name) ?? [])
265263
);
266264

267265
if (!notebookName) {

0 commit comments

Comments
 (0)