🐛 📝 (Pinned documentations + CHANGELOG) - Pinned documentations are not correctly removed when unpinned + Add publication date in CHANGELOG

Author: baptistelechat

CHANGELOG.md

@@ -5,7 +5,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog],
 and this project adheres to [Semantic Versioning].
 
-## Unreleased
+<!-- ## Unreleased -->
+
+## [0.3.0] - 2024-08-27
 
 ### Added
 
@@ -74,3 +76,5 @@ and this project adheres to [Semantic Versioning].
 
 [0.1.0]: https://github.com/baptistelechat/codeX/releases/tag/v0.1.0
 [0.2.0]: https://github.com/baptistelechat/codeX/releases/tag/v0.2.0
+[0.2.1]: https://github.com/baptistelechat/codeX/releases/tag/v0.2.1
+[0.3.0]: https://github.com/baptistelechat/codeX/releases/tag/v0.3.0

src/app/Sidebar/sidebar.ts

@@ -71,15 +71,15 @@ const loadDocumentations = (
     pinnedDocumentations,
     favoriteDocumentations,
     hideDocumentations,
-    searchMode
+    true
   );
 
   documentations = sortDocumentations(
     newDocumentations,
     pinnedDocumentations,
     favoriteDocumentations,
     hideDocumentations,
-    searchMode
+    false
   );
 
   const container = document.getElementById("documentation-container");

src/lib/utils/pinnedDocumentations/togglePinned.ts

@@ -1,4 +1,3 @@
-import getAllDocumentations from "../documentation/getAllDocumentations";
 import getDocumentationName from "../getDocumentationName";
 import { DocumentationViewProvider } from "../provider/DocumentationViewProvider";
 import { showInformationMessage } from "../showMessage";
@@ -7,15 +6,60 @@ const togglePinned = async (
   provider: DocumentationViewProvider,
   documentationId: string
 ) => {
-  const index = provider._pinnedDocumentations.indexOf(documentationId);
+  const pinnedDocumentationIndex =
+    provider._pinnedDocumentations.indexOf(documentationId);
+
+  const documentationIndex = provider._documentations.findIndex(
+    (documentation) => documentation.id === documentationId
+  );
+
+  const searchDocumentationIndex = provider._searchDocumentations.findIndex(
+    (documentation) => documentation.id === documentationId
+  );
+
+  if (pinnedDocumentationIndex !== -1) {
+    provider._pinnedDocumentations.splice(pinnedDocumentationIndex, 1);
+
+    if (documentationIndex !== -1) {
+      provider._documentations[documentationIndex].isPinned = false;
+    }
+
+    if (searchDocumentationIndex !== -1) {
+      provider._searchDocumentations[searchDocumentationIndex].isPinned = false;
+    }
 
-  if (index !== -1) {
-    provider._pinnedDocumentations.splice(index, 1);
     showInformationMessage(
       `${getDocumentationName(provider, documentationId)} unpinned.`
     );
   } else {
     provider._pinnedDocumentations.push(documentationId);
+
+    if (documentationIndex !== -1) {
+      provider._documentations[documentationIndex].isPinned = true;
+    }
+
+    if (searchDocumentationIndex !== -1) {
+      provider._searchDocumentations[searchDocumentationIndex].isPinned = true;
+
+      provider._documentations.push(
+        provider._searchDocumentations[searchDocumentationIndex]
+      );
+
+      const seenIds = new Set();
+
+      const uniqueDocumentations = provider._documentations.filter(
+        (documentation) => {
+          if (seenIds.has(documentation.id)) {
+            return false;
+          }
+          seenIds.add(documentation.id);
+          return true;
+        }
+      );
+
+      provider._documentations = uniqueDocumentations;
+    }
+
     showInformationMessage(
       `${getDocumentationName(provider, documentationId)} pinned for later.`
     );
@@ -24,27 +68,6 @@ const togglePinned = async (
   await provider.savePinnedDocumentations();
 
   if (provider._view) {
-    const pinnedDocumentations = await getAllDocumentations(provider, [
-      ...provider._pinnedDocumentations,
-      ...provider._searchDocumentations
-        .filter((documentation) => documentation.isPinned)
-        .map((documentation) => documentation.id),
-    ]);
-
-    const seenIds = new Set();
-
-    const uniqueDocumentations = provider._documentations.filter(
-      (documentation) => {
-        if (seenIds.has(documentation.id)) {
-          return false;
-        }
-        seenIds.add(documentation.id);
-        return true;
-      }
-    );
-
-    provider._documentations = uniqueDocumentations;
-
     provider._view.webview.postMessage({
       type: "setDocumentations",
       documentations: provider._documentations,