Remove unnecessary double popup for replace

748005b903600fab5f7718e77e145781a5044afa

Wolfsblvt <wolfsblvt@gmail.com>

public/scripts/bookmarks.js+4 -9 Viewed

@@ -57,10 +57,10 @@ async function getExistingChatNames() {
57	57	}
58	58	}
59	59
60	60	async function getBookmarkName({ isReplace = false, forceName = null } = {}) {
61	61	const chatNames = await getExistingChatNames();
62	62
63	63	const body = await renderTemplateAsync('createCheckpoint', { isReplace: isReplace });
64	64	let name = forceName \|\| await Popup.show.input('Create Checkpoint', body);
65	65	if (name === null) {
66	66	return null;

         lastMes.extra = {};
+    }
 -    if (lastMes.extra.bookmark_link && !forceName) {
 +    const isReplace = lastMes.extra.bookmark_link;
 -        const confirm = await Popup.show.confirm('Replace Checkpoint', 'Checkpoint for the last message already exists.<br />Would you like to replace it?');
 -        if (!confirm) {
 -            return null;
 -        }
 -    }
     let name = await getBookmarkName({ isReplace: isReplace, forceName: forceName });
     if (!name) {
         return null;
+    }

public/scripts/templates/createCheckpoint.html+7 -0 Viewed

 +<div>
     <span class="margin-right-10px">Enter Checkpoint Name:</span><small>(Leave empty to auto-generate)</small>
 +</div>
 +{{#if isReplace}}
 +<div class="m-t-1">
 +    <small>The currently existing checkpoint will be unlinked and replaced with the new checkpoint, but can still be found in the Chat Management.</small>
 +</div>
 +{{/if}}