[PR #15] [MERGED] Terminology fixes #25

Closed
opened 2026-10-15 18:07:56 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/SillyTavern/SillyTavern-Timelines/pull/15
Author: @Technologicat
Created: 1/25/2024
Status: ✅ Merged
Merged: 1/25/2024
Merged by: @Cohee1207

Base: master ← Head: terminology-fixes


📝 Commits (5)

  • 68d7914 mention slash command in README
  • 3a688cc whitespace fix
  • 489076b consistent name for extension: Timelines (plural!)
  • 7e54978 old name "bookmark" -> current name "checkpoint" to match ST 1.11.3
  • 256a366 revert the check, add comment

📊 Changes

5 files changed (+33 additions, -19 deletions)

View changed files

📝 README.md (+13 -6)
📝 index.js (+1 -1)
📝 timeline.html (+2 -2)
📝 tl_node_data.js (+13 -6)
📝 tl_style.js (+4 -4)

📄 Description

This is getting ridiculous, but here's another PR. :)

As per https://github.com/SillyTavern/SillyTavern/issues/1739,

  • Extension name is now consistently Timelines (plural!) also in the extension settings panel
  • Terminology change to match ST 1.11.3: "bookmark" → "checkpoint"
    • Even the ST codebase still calls them bookmarks internally, so I changed only the user-visible strings and comments.
  • Fix a check that didn't trigger after ST itself changed this terminology EDIT: Reverted as per review.
  • Mention the /tl slash command in the README
  • Fix whitespace in README

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/SillyTavern/SillyTavern-Timelines/pull/15 **Author:** [@Technologicat](https://github.com/Technologicat) **Created:** 1/25/2024 **Status:** ✅ Merged **Merged:** 1/25/2024 **Merged by:** [@Cohee1207](https://github.com/Cohee1207) **Base:** `master` ← **Head:** `terminology-fixes` --- ### 📝 Commits (5) - [`68d7914`](https://github.com/SillyTavern/SillyTavern-Timelines/commit/68d79143d7281dc47bcb8f388c20413e3d16a630) mention slash command in README - [`3a688cc`](https://github.com/SillyTavern/SillyTavern-Timelines/commit/3a688cc14dce50c1f1e7a44583689d3a1af8f797) whitespace fix - [`489076b`](https://github.com/SillyTavern/SillyTavern-Timelines/commit/489076b0412630e4429ba67361e80ce9057abf39) consistent name for extension: Timelines (plural!) - [`7e54978`](https://github.com/SillyTavern/SillyTavern-Timelines/commit/7e549782611ccfc003034fa59056627ea2754373) old name "bookmark" -> current name "checkpoint" to match ST 1.11.3 - [`256a366`](https://github.com/SillyTavern/SillyTavern-Timelines/commit/256a3661b2b870b56494250d45a3797d08c92c80) revert the check, add comment ### 📊 Changes **5 files changed** (+33 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+13 -6) 📝 `index.js` (+1 -1) 📝 `timeline.html` (+2 -2) 📝 `tl_node_data.js` (+13 -6) 📝 `tl_style.js` (+4 -4) </details> ### 📄 Description This is getting ridiculous, but here's another PR. :) As per https://github.com/SillyTavern/SillyTavern/issues/1739, - Extension name is now consistently *Timelines* (plural!) also in the extension settings panel - Terminology change to match ST 1.11.3: "bookmark" → "checkpoint" - Even the ST codebase still calls them bookmarks internally, so I changed only the user-visible strings and comments. - ~Fix a check that didn't trigger after ST itself changed this terminology~ **EDIT**: *Reverted as per review.* - Mention the `/tl` slash command in the README - Fix whitespace in README --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-15 18:07:56 +00:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
SillyTavern/SillyTavern-Timelines#25
No description provided.