Disable changelog menu opt if curr ver is not available

src/auto-updater/index.js

@@ -28,6 +28,8 @@ const {
   addOnceProcEventHandler
 } = require('../window-event-manager')
 
+const MENU_ITEM_IDS = require('../create-menu/menu.item.ids')
+
 const isAutoUpdateDisabled = parseEnvValToBool(process.env.IS_AUTO_UPDATE_DISABLED)
 
 const fontsStyle = fs.readFileSync(path.join(
@@ -219,8 +221,12 @@ const _switchMenuItem = (opts = {}) => {
     isCheckMenuItemDisabled,
     isInstallMenuItemVisible
   } = { ...opts }
-  const checkMenuItem = _getUpdateMenuItemById('CHECK_UPDATE_MENU_ITEM')
-  const installMenuItem = _getUpdateMenuItemById('INSTALL_UPDATE_MENU_ITEM')
+  const checkMenuItem = _getUpdateMenuItemById(
+    MENU_ITEM_IDS.CHECK_UPDATE_MENU_ITEM
+  )
+  const installMenuItem = _getUpdateMenuItemById(
+    MENU_ITEM_IDS.INSTALL_UPDATE_MENU_ITEM
+  )
 
   if (
     !checkMenuItem ||

src/changelog-manager/manage-changelog.js

@@ -23,14 +23,21 @@ module.exports = async () => {
       return
     }
 
-    const isShown = await showChangelog({ version })
+    const {
+      error,
+      isShown
+    } = await showChangelog({ version })
+
+    if (!isShown) {
+      return
+    }
 
     const isSaved = await configsKeeper
       .saveConfigs({ shownChangelogVer: version })
 
     if (
       isSaved &&
-      isShown
+      !error
     ) {
       return
     }

src/changelog-manager/show-changelog.js

@@ -1,14 +1,24 @@
 'use strict'
 
+const { Menu } = require('electron')
 const path = require('path')
 const parseChangelog = require('changelog-parser')
 const { rootPath } = require('electron-root-path')
 
 const getDebugInfo = require('../helpers/get-debug-info')
 const showDocs = require('../show-docs')
 
+const MENU_ITEM_IDS = require('../create-menu/menu.item.ids')
+
 const changelogPath = path.join(rootPath, 'CHANGELOG.md')
 
+const disableShowChangelogMenuItem = () => {
+  const menuItem = Menu.getApplicationMenu()
+    ?.getMenuItemById(MENU_ITEM_IDS.SHOW_CHANGE_LOG_MENU_ITEM) ?? {}
+
+  menuItem.enabled = false
+}
+
 module.exports = async (params = {}) => {
   try {
     const version = params?.version ?? getDebugInfo()?.version
@@ -23,7 +33,12 @@ module.exports = async (params = {}) => {
       !Array.isArray(mdEntries?.versions) ||
       mdEntries?.versions.length === 0
     ) {
-      return true
+      disableShowChangelogMenuItem()
+
+      return {
+        error: null,
+        isShown: false
+      }
     }
 
     const mdEntry = mdEntries.versions
@@ -33,7 +48,12 @@ module.exports = async (params = {}) => {
       !mdEntry?.title ||
       !mdEntry?.body
     ) {
-      return true
+      disableShowChangelogMenuItem()
+
+      return {
+        error: null,
+        isShown: false
+      }
     }
 
     const mdTitle = `# ${mdEntries.title}`
@@ -45,10 +65,16 @@ module.exports = async (params = {}) => {
       mdDoc
     })
 
-    return true
-  } catch (err) {
-    console.error(err)
+    return {
+      error: null,
+      isShown: true
+    }
+  } catch (error) {
+    console.error(error)
 
-    return false
+    return {
+      error,
+      isShown: false
+    }
   }
 }

src/create-menu.js → src/create-menu/index.js

@@ -5,23 +5,25 @@ const electron = require('electron')
 const { app, Menu } = electron
 const isMac = process.platform === 'darwin'
 
-const exportDB = require('./export-db')
-const importDB = require('./import-db')
-const removeDB = require('./remove-db')
-const restoreDB = require('./restore-db')
-const backupDB = require('./backup-db')
-const changeReportsFolder = require('./change-reports-folder')
-const changeSyncFrequency = require('./change-sync-frequency')
-const triggerElectronLoad = require('./trigger-electron-load')
-const showAboutModalDialog = require('./show-about-modal-dialog')
+const exportDB = require('../export-db')
+const importDB = require('../import-db')
+const removeDB = require('../remove-db')
+const restoreDB = require('../restore-db')
+const backupDB = require('../backup-db')
+const changeReportsFolder = require('../change-reports-folder')
+const changeSyncFrequency = require('../change-sync-frequency')
+const triggerElectronLoad = require('../trigger-electron-load')
+const showAboutModalDialog = require('../show-about-modal-dialog')
 const {
   checkForUpdates,
   quitAndInstall
-} = require('./auto-updater')
-const { manageNewGithubIssue } = require('./error-manager')
-const showDocs = require('./show-docs')
-const { showChangelog } = require('./changelog-manager')
-const parseEnvValToBool = require('./helpers/parse-env-val-to-bool')
+} = require('../auto-updater')
+const { manageNewGithubIssue } = require('../error-manager')
+const showDocs = require('../show-docs')
+const { showChangelog } = require('../changelog-manager')
+const parseEnvValToBool = require('../helpers/parse-env-val-to-bool')
+
+const MENU_ITEM_IDS = require('./menu.item.ids')
 
 const isAutoUpdateDisabled = parseEnvValToBool(process.env.IS_AUTO_UPDATE_DISABLED)
 
@@ -137,20 +139,20 @@ module.exports = ({
       submenu: [
         {
           label: 'Open new GitHub issue',
-          id: 'REPORT_BUG_MENU_ITEM',
+          id: MENU_ITEM_IDS.REPORT_BUG_MENU_ITEM,
           click: manageNewGithubIssue
         },
         { type: 'separator' },
         {
           label: 'Check for updates',
           enabled: !isAutoUpdateDisabled,
-          id: 'CHECK_UPDATE_MENU_ITEM',
+          id: MENU_ITEM_IDS.CHECK_UPDATE_MENU_ITEM,
           click: checkForUpdates()
         },
         {
           label: 'Quit and install updates',
           visible: false,
-          id: 'INSTALL_UPDATE_MENU_ITEM',
+          id: MENU_ITEM_IDS.INSTALL_UPDATE_MENU_ITEM,
           click: quitAndInstall()
         },
         { type: 'separator' },
@@ -161,6 +163,7 @@ module.exports = ({
         },
         {
           label: 'Changelog',
+          id: MENU_ITEM_IDS.SHOW_CHANGE_LOG_MENU_ITEM,
           click: () => showChangelog()
         },
         ...(isMac

src/create-menu/menu.item.ids.js

@@ -0,0 +1,8 @@
+'use strict'
+
+module.exports = {
+  REPORT_BUG_MENU_ITEM: 'REPORT_BUG_MENU_ITEM',
+  CHECK_UPDATE_MENU_ITEM: 'CHECK_UPDATE_MENU_ITEM',
+  INSTALL_UPDATE_MENU_ITEM: 'INSTALL_UPDATE_MENU_ITEM',
+  SHOW_CHANGE_LOG_MENU_ITEM: 'SHOW_CHANGE_LOG_MENU_ITEM'
+}

src/error-manager/index.js

@@ -14,6 +14,8 @@ const openNewGithubIssue = require('./open-new-github-issue')
 const collectLogs = require('./collect-logs')
 const getDebugInfo = require('../helpers/get-debug-info')
 
+const MENU_ITEM_IDS = require('../create-menu/menu.item.ids')
+
 let _isLocked = false
 let _isIssueAutoManagerLocked = false
 let caughtError
@@ -97,7 +99,7 @@ const _getReportBugMenuItem = () => {
     return {}
   }
 
-  return menu.getMenuItemById('REPORT_BUG_MENU_ITEM') || {}
+  return menu.getMenuItemById(MENU_ITEM_IDS.REPORT_BUG_MENU_ITEM) || {}
 }
 
 const _lockIssueManager = () => {