refactor: apply code review comments

dhis2 · Mar 6, 2024 · 60e79f8 · 60e79f8
1 parent 70363f7
commit 60e79f8
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 13 deletions.
diff --git a/src/hooks/useTasks.js b/src/hooks/useTasks.js
@@ -48,7 +48,6 @@ const defaultRefetchPeriod = 2000
 const createFetchEvents =
     (engine, setTasks, fetchSummary) => (type, id, task) => {
         const fetchEvents = async () => {
-            console.debug('[fetch event]', type, id, task)
             if (task.completed) {
                 return
             }

diff --git a/src/pages/EventExport/EventExport.js b/src/pages/EventExport/EventExport.js
@@ -41,7 +41,7 @@ const { Form } = ReactFinalForm
 // PAGE INFO
 export const PAGE_NAME = i18n.t('Event export')
 export const PAGE_DESCRIPTION = i18n.t(
-    'Export event data for programs, stages and tracked entities to JSON, CSV, or DXF2 format.'
+    'Export event data for programs, stages and tracked entities to JSON or CSV format.'
 )
 const PAGE_ICON = <EventIcon />
 

diff --git a/src/pages/EventImport/EventImport.js b/src/pages/EventImport/EventImport.js
@@ -34,7 +34,7 @@ import { onImport } from './form-helper.js'
 // PAGE INFO
 export const PAGE_NAME = i18n.t('Event import')
 export const PAGE_DESCRIPTION = i18n.t(
-    'Import event data for programs, stages and tracked entities to JSON, CSV, or DXF2 format.'
+    'Import event data for programs, stages and tracked entities from JSON or CSV format.'
 )
 const PAGE_ICON = <EventIcon />
 
@@ -95,7 +95,7 @@ const EventImport = () => {
                         <BasicOptions>
                             <FileUpload
                                 helpText={i18n.t(
-                                    'Supported file types: JSON, CSV, and DXF2.',
+                                    'Supported file types: JSON and CSV.',
                                     {
                                         nsSeparator: '>',
                                     }

diff --git a/src/pages/TEIExport/TEIExport.js b/src/pages/TEIExport/TEIExport.js
@@ -54,7 +54,7 @@ const { Form } = ReactFinalForm
 // PAGE INFO
 export const PAGE_NAME = i18n.t('Tracked entities export')
 export const PAGE_DESCRIPTION = i18n.t(
-    'Export tracked entities in JSON, CSV, or DXF2 format.'
+    'Export tracked entities in JSON or CSV format.'
 )
 const PAGE_ICON = <TEIIcon />
 

diff --git a/src/pages/TEIExport/form-helper.js b/src/pages/TEIExport/form-helper.js
@@ -70,8 +70,6 @@ const valuesToParams = (
             minParams.programStatus = programStatus
         }
 
-        console.log('>>>>>', followup, minParams.followup)
-
         if (followup !== 'ALL') {
             minParams.followup = followup
         }

diff --git a/src/pages/TEIImport/TEIImport.js b/src/pages/TEIImport/TEIImport.js
@@ -14,7 +14,6 @@ import {
 import {
     FileUpload,
     Format,
-    formatNoCsvOptions,
     defaultFormatOption,
     Identifier,
     defaultIdentifierOption,
@@ -46,6 +45,7 @@ import {
     defaultIdSchemeOption,
     OrgUnitIdScheme,
     defaultOrgUnitIdSchemeOption,
+    formatNoXmlNoCsvOptions,
 } from '../../components/Inputs/index.js'
 import { TaskContext, getNewestTask } from '../../contexts/index.js'
 import { getPrevJobDetails, getInitialBoolValue } from '../../utils/helper.js'
@@ -56,7 +56,7 @@ const { Form } = ReactFinalForm
 // PAGE INFO
 export const PAGE_NAME = i18n.t('Tracked entities import')
 export const PAGE_DESCRIPTION = i18n.t(
-    'Import tracked entities using JSON or DXF2 format.'
+    'Import tracked entities using JSON format.'
 )
 const PAGE_ICON = <TEIIcon />
 
@@ -134,14 +134,14 @@ const TEIImport = () => {
                         <BasicOptions>
                             <FileUpload
                                 helpText={i18n.t(
-                                    'Supported file types: JSON and DXF2.',
+                                    'Supported file types: JSON.',
                                     {
                                         nsSeparator: '>',
                                     }
                                 )}
                             />
                             <Format
-                                availableFormats={formatNoCsvOptions}
+                                availableFormats={formatNoXmlNoCsvOptions}
                                 type="import"
                             />
                             <Identifier />

diff --git a/src/pages/TEIImport/form-helper.js b/src/pages/TEIImport/form-helper.js
@@ -49,7 +49,7 @@ const onImport =
             .filter((s) => s != '')
             .join('&')
         const url = `${apiBaseUrl}?${params}`
-        console.debug('[TEI import url]', url)
+
         try {
             await uploadFile({
                 url,

diff --git a/src/utils/helper.js b/src/utils/helper.js
@@ -104,7 +104,6 @@ const uploadFile = ({
                 onResponse: (response) => {
                     const { error, id, msg, typeReports } = response
                     let entry
-                    console.debug('response from uploading file', response)
                     if (!isAsync) {
                         // we are done
                         entry = {