Prepare pre-release

app/src/actions/clearTopic.ts

@@ -5,14 +5,14 @@ import { makePublishEvent, rendererEvents } from '../../../events'
 import { moveSelectionUpOrDownwards } from './visibleTreeTraversal'
 import { globalActions } from '.'
 
-export const clearTopic = (topic: q.TreeNode<any>, recursive: boolean, subtopicClearLimit = 50) => async (
+export const clearTopic = (topic: q.TreeNode<any>, recursive: boolean) => async (
   dispatch: Dispatch<any>,
   getState: () => AppState
 ) => {
+  const topicsForPurging = recursive ? [topic, ...topic.childTopics()] : [topic]
+
   if (recursive) {
     const topicCount = topic.childTopicCount()
-    const deleteLimitMessage =
-      topicCount > subtopicClearLimit ? ` You can only delete ${subtopicClearLimit} child topics at once.` : ''
 
     const topicDelta = topic.hasMessage() ? -1 : 0
     const childTopicsMessage =
@@ -23,7 +23,7 @@ export const clearTopic = (topic: q.TreeNode<any>, recursive: boolean, subtopicC
     const confirmed = await dispatch(
       globalActions.requestConfirmation(
         'Confirm delete',
-        `Do you want to delete "${topic.path()}"${childTopicsMessage}?${deleteLimitMessage}`
+        `Do you want to clear "${topic.path()}"${childTopicsMessage}?\n\nThis function will send an empty payload (QoS 0, retain) to this and every subtopic, clearing retained topics in the process. Only use this function if you know what you are doing.`
       )
     )
     if (!confirmed) {
@@ -38,27 +38,18 @@ export const clearTopic = (topic: q.TreeNode<any>, recursive: boolean, subtopicC
     return
   }
   const publishEvent = makePublishEvent(connectionId)
-  const mqttMessage = {
-    topic: topic.path(),
-    payload: null,
-    retain: true,
-    qos: 0 as 0,
-  }
-  rendererEvents.emit(publishEvent, mqttMessage)
-  if (recursive) {
-    topic
-      .childTopics()
-      .filter(topic => Boolean(topic.message && topic.message.value))
-      .slice(0, subtopicClearLimit)
-      .forEach((topic, idx) => {
-        const mqttMessage = {
-          topic: topic.path(),
-          payload: null,
-          retain: true,
-          qos: 0 as 0,
-        }
-        // Rate limit deletion
-        setTimeout(() => rendererEvents.emit(publishEvent, mqttMessage), 20 * idx)
-      })
-  }
+
+  topicsForPurging
+    .filter(t => t.path() !== '' && t.hasMessage())
+    .map(t => t.path())
+    .forEach((path, idx) => {
+      const mqttMessage = {
+        topic: path,
+        payload: null,
+        retain: true,
+        qos: 0 as 0,
+      }
+      // Rate limit deletion
+      setTimeout(() => rendererEvents.emit(publishEvent, mqttMessage), 20 * idx)
+    })
 }

app/src/components/ConfirmationDialog.tsx

@@ -42,13 +42,15 @@ function ConfirmationDialog(props: { confirmationRequests: Array<ConfirmationReq
     >
       <DialogTitle id="alert-dialog-title">{request.title}</DialogTitle>
       <DialogContent>
-        <DialogContentText id="alert-dialog-description">{request.inquiry}</DialogContentText>
+        <DialogContentText id="alert-dialog-description" style={{ whiteSpace: 'pre-wrap' }}>
+          {request.inquiry}
+        </DialogContentText>
       </DialogContent>
       <DialogActions>
         <Button ref={yesRef as any} variant="contained" onClick={confirm} color="primary" autoFocus>
           Yes
         </Button>
-        <Button ref={noRef as any} variant="contained" onClick={reject} color="secondary">
+        <Button ref={noRef as any} variant="contained" onClick={reject} color="secondary" style={{ marginLeft: '8px' }}>
           No
         </Button>
       </DialogActions>

app/src/components/Sidebar/HistoryDrawer.tsx

@@ -17,7 +17,7 @@ interface Props {
   contentTypeIndicator?: JSX.Element
   autoOpen?: boolean
   theme: Theme
-  children: any
+  children?: any
 }
 
 function HistoryDrawer(props: Props) {

app/src/components/Sidebar/TopicPanel/RecursiveTopicDeleteButton.tsx

@@ -13,25 +13,23 @@ export const RecursiveTopicDeleteButton = (props: {
       if (props.node) {
         event.stopPropagation()
         event.preventDefault()
-        props.deleteTopicAction(props.node, true, deleteLimit)
+        props.deleteTopicAction(props.node, true, Infinity)
       }
     },
     [props.node]
   )
+
   if (!props.node) {
     return null
   }
-  const deleteLimit = 50
+
   const topicCount = props.node ? props.node.childTopicCount() : 0
   if (topicCount === 0 || (props.node.message && topicCount === 1)) {
     return null
   }
   return (
-    <Badge
-      badgeContent={<span style={{ whiteSpace: 'nowrap' }}>{topicCount >= deleteLimit ? '50+' : topicCount}</span>}
-      color="secondary"
-    >
-      <CustomIconButton onClick={onClick} tooltip={`Deletes up to ${deleteLimit} sub-topics with a single click`}>
+    <Badge badgeContent={<span style={{ whiteSpace: 'nowrap' }}>{topicCount}</span>} color="secondary">
+      <CustomIconButton onClick={onClick} tooltip={`Deletes ${topicCount} sub-topics with a single click`}>
         <Delete color="action" />
       </CustomIconButton>
     </Badge>

app/src/components/Sidebar/TopicPanel/TopicPanel.tsx

@@ -14,12 +14,12 @@ const TopicPanel = (props: { node?: q.TreeNode<any>; actions: typeof sidebarActi
   console.log(node && node.path())
   const copyTopic = node ? <Copy value={node.path()} /> : null
 
-  const deleteTopic = useCallback((topic?: q.TreeNode<any>, recursive: boolean = false, maxCount = 50) => {
+  const deleteTopic = useCallback((topic?: q.TreeNode<any>, recursive: boolean = false) => {
     if (!topic) {
       return
     }
 
-    props.actions.clearTopic(topic, recursive, maxCount)
+    props.actions.clearTopic(topic, recursive)
   }, [])
 
   return useMemo(

app/src/components/Tree/TreeNode/effects/useDeleteKeyCallback.tsx

@@ -9,7 +9,7 @@ export function useDeleteKeyCallback(topic: q.TreeNode<any>, actions: typeof tre
       if (event.keyCode === KeyCodes.delete || event.keyCode === KeyCodes.backspace) {
         event.stopPropagation()
         event.preventDefault()
-        actions.clearTopic(topic, true, 50)
+        actions.clearTopic(topic, true)
       }
     },
     [topic]

package.json

@@ -1,6 +1,6 @@
 {
   "name": "MQTT-Explorer",
-  "version": "0.3.3",
+  "version": "0.3.6-no-delete-limit",
   "description": "Explore your message queues",
   "main": "dist/src/electron.js",
   "scripts": {

src/registerCrashReporter.ts

@@ -4,7 +4,7 @@ export function registerCrashReporter() {
   crashReporter.start({
     productName: 'MQTT Explorer',
     companyName: 'thomasnordquist',
-    submitURL: 'http://localhost:3000/app/crash/mqttexplorer',
+    submitURL: 'http://app-telemetry.t7n.de/app/crash/mqttexplorer',
     uploadToServer: true,
   })
 }