From 1835663afa6ddc970e5f695fdaef645e3ebff5d6 Mon Sep 17 00:00:00 2001
From: Birk Johansson <birk@dhis2.org>
Date: Thu, 22 Sep 2022 13:27:34 +0200
Subject: [PATCH 1/7] fix(description-editor): fix some styling issues

---
 client/src/pages/UserAppEdit/DescriptionEditor.js   | 13 ++++++++-----
 client/src/pages/UserAppEdit/UserAppEdit.module.css | 12 ++++++++++++
 2 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/client/src/pages/UserAppEdit/DescriptionEditor.js b/client/src/pages/UserAppEdit/DescriptionEditor.js
index 7068c0b41..5596ec7a7 100644
--- a/client/src/pages/UserAppEdit/DescriptionEditor.js
+++ b/client/src/pages/UserAppEdit/DescriptionEditor.js
@@ -5,6 +5,7 @@ import { useField } from 'react-final-form'
 import AppDescription from '../../components/AppDescription/AppDescription'
 import styles from './UserAppEdit.module.css'
 import MarkdownEditor from '../../components/MarkdownEditor/MarkdownEditor'
+import cx from 'classnames'
 
 const tabs = {
     WRITE: 'WRITE',
@@ -38,9 +39,11 @@ const DescriptionEditor = ({ description }) => {
                     Preview
                 </Tab>
             </TabBar>
-            {selectedTab === tabs.WRITE && (
-                <WriteContent description={description} />
-            )}
+
+            <WriteContent
+                description={description}
+                hidden={selectedTab !== tabs.WRITE}
+            />
             {selectedTab === tabs.PREVIEW && <PreviewContent />}
             <Help className={styles.helpText}>
                 A good app description helps users of the App Hub quickly
@@ -55,9 +58,9 @@ DescriptionEditor.propTypes = {
     description: PropTypes.string,
 }
 
-const WriteContent = ({ description }) => {
+const WriteContent = ({ description, hidden }) => {
     return (
-        <div className={styles.writeContent}>
+        <div className={cx(styles.writeContent, { [styles.hidden]: hidden })}>
             <MarkdownEditor
                 name="description"
                 initialValue={description}
diff --git a/client/src/pages/UserAppEdit/UserAppEdit.module.css b/client/src/pages/UserAppEdit/UserAppEdit.module.css
index 4eb1dc067..5ae912302 100644
--- a/client/src/pages/UserAppEdit/UserAppEdit.module.css
+++ b/client/src/pages/UserAppEdit/UserAppEdit.module.css
@@ -43,3 +43,15 @@
 .writeContent {
     margin-top: 8px;
 }
+
+.writeContent :global(textarea) {
+    min-height: 48px;
+}
+
+.writeContent :global(p.error) {
+    margin-left: 8px;
+}
+
+.hidden {
+    display: none;
+}

From 2b9f82594f0eb5e533a1b33e4ac885e980396aad Mon Sep 17 00:00:00 2001
From: Birk Johansson <birk@dhis2.org>
Date: Mon, 26 Sep 2022 23:26:29 +0200
Subject: [PATCH 2/7] chore: check run CI


From 3f8c2bc5bf8a984b918392fd8f69064407446898 Mon Sep 17 00:00:00 2001
From: "@dhis2-bot" <apps@dhis2.org>
Date: Tue, 15 Nov 2022 10:48:16 +0000
Subject: [PATCH 3/7] chore(release): cut 2.31.1 [skip release]

## [2.31.1](https://github.com/dhis2/app-hub/compare/v2.31.0...v2.31.1) (2022-11-15)

### Bug Fixes

* **description-editor:** fix some styling issues ([1835663](https://github.com/dhis2/app-hub/commit/1835663afa6ddc970e5f695fdaef645e3ebff5d6))
---
 CHANGELOG.md        | 7 +++++++
 client/package.json | 2 +-
 package.json        | 2 +-
 server/package.json | 2 +-
 tools/package.json  | 2 +-
 5 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 749ee83af..40faa7f8e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,10 @@
+## [2.31.1](https://github.com/dhis2/app-hub/compare/v2.31.0...v2.31.1) (2022-11-15)
+
+
+### Bug Fixes
+
+* **description-editor:** fix some styling issues ([1835663](https://github.com/dhis2/app-hub/commit/1835663afa6ddc970e5f695fdaef645e3ebff5d6))
+
 # [2.31.0](https://github.com/dhis2/app-hub/compare/v2.30.2...v2.31.0) (2022-09-21)
 
 
diff --git a/client/package.json b/client/package.json
index 19b732664..8a37e6b84 100644
--- a/client/package.json
+++ b/client/package.json
@@ -1,6 +1,6 @@
 {
     "name": "client",
-    "version": "2.31.0",
+    "version": "2.31.1",
     "description": "The App Hub Client",
     "repository": "https://github.com/dhis2/app-hub",
     "author": "Birk Johansson <birk@dhis2.org>",
diff --git a/package.json b/package.json
index 6a4c271c5..d2da6faa5 100644
--- a/package.json
+++ b/package.json
@@ -36,5 +36,5 @@
     "cypress-cucumber-preprocessor": {
         "nonGlobalStepDefinitions": true
     },
-    "version": "2.31.0"
+    "version": "2.31.1"
 }
diff --git a/server/package.json b/server/package.json
index 079177891..e19fe0213 100644
--- a/server/package.json
+++ b/server/package.json
@@ -1,6 +1,6 @@
 {
     "name": "server",
-    "version": "2.31.0",
+    "version": "2.31.1",
     "description": "The App Hub Server",
     "main": "src/main.js",
     "repository": "https://github.com/dhis2/app-hub",
diff --git a/tools/package.json b/tools/package.json
index 63c8ccf6b..e95948841 100644
--- a/tools/package.json
+++ b/tools/package.json
@@ -1,6 +1,6 @@
 {
     "name": "tools",
-    "version": "2.31.0",
+    "version": "2.31.1",
     "description": "",
     "main": "clone.js",
     "repository": "https://github.com/dhis2/app-hub",

From 24bd6bc55221d3fbb06d55dd637fc2d79738a4ab Mon Sep 17 00:00:00 2001
From: Hendrik de Graaf <HendrikThePendric@users.noreply.github.com>
Date: Tue, 15 Nov 2022 17:06:47 +0100
Subject: [PATCH 4/7] fix(add app): allow clearing optional select field (#694)

---
 client/src/pages/UserAppUpload/UserAppUpload.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/client/src/pages/UserAppUpload/UserAppUpload.js b/client/src/pages/UserAppUpload/UserAppUpload.js
index f58a1b5b6..c629de3fc 100644
--- a/client/src/pages/UserAppUpload/UserAppUpload.js
+++ b/client/src/pages/UserAppUpload/UserAppUpload.js
@@ -314,6 +314,7 @@ const UserAppUpload = ({ user }) => {
                                 className={styles.field}
                                 validate={maxDhisVersionValidator}
                                 options={dhisVersionOptions}
+                                clearable
                             />
                         </section>
 

From f8da6907b582ed18c72c7176c1689e35bc3a1d42 Mon Sep 17 00:00:00 2001
From: "@dhis2-bot" <apps@dhis2.org>
Date: Tue, 15 Nov 2022 16:07:22 +0000
Subject: [PATCH 5/7] chore(release): cut 2.31.2 [skip release]

## [2.31.2](https://github.com/dhis2/app-hub/compare/v2.31.1...v2.31.2) (2022-11-15)

### Bug Fixes

* **add app:** allow clearing optional select field ([#694](https://github.com/dhis2/app-hub/issues/694)) ([24bd6bc](https://github.com/dhis2/app-hub/commit/24bd6bc55221d3fbb06d55dd637fc2d79738a4ab))
---
 CHANGELOG.md        | 7 +++++++
 client/package.json | 2 +-
 package.json        | 2 +-
 server/package.json | 2 +-
 tools/package.json  | 2 +-
 5 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 40faa7f8e..9d0b2f17f 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,10 @@
+## [2.31.2](https://github.com/dhis2/app-hub/compare/v2.31.1...v2.31.2) (2022-11-15)
+
+
+### Bug Fixes
+
+* **add app:** allow clearing optional select field ([#694](https://github.com/dhis2/app-hub/issues/694)) ([24bd6bc](https://github.com/dhis2/app-hub/commit/24bd6bc55221d3fbb06d55dd637fc2d79738a4ab))
+
 ## [2.31.1](https://github.com/dhis2/app-hub/compare/v2.31.0...v2.31.1) (2022-11-15)
 
 
diff --git a/client/package.json b/client/package.json
index 8a37e6b84..004631813 100644
--- a/client/package.json
+++ b/client/package.json
@@ -1,6 +1,6 @@
 {
     "name": "client",
-    "version": "2.31.1",
+    "version": "2.31.2",
     "description": "The App Hub Client",
     "repository": "https://github.com/dhis2/app-hub",
     "author": "Birk Johansson <birk@dhis2.org>",
diff --git a/package.json b/package.json
index d2da6faa5..fbc3657ef 100644
--- a/package.json
+++ b/package.json
@@ -36,5 +36,5 @@
     "cypress-cucumber-preprocessor": {
         "nonGlobalStepDefinitions": true
     },
-    "version": "2.31.1"
+    "version": "2.31.2"
 }
diff --git a/server/package.json b/server/package.json
index e19fe0213..89c131a52 100644
--- a/server/package.json
+++ b/server/package.json
@@ -1,6 +1,6 @@
 {
     "name": "server",
-    "version": "2.31.1",
+    "version": "2.31.2",
     "description": "The App Hub Server",
     "main": "src/main.js",
     "repository": "https://github.com/dhis2/app-hub",
diff --git a/tools/package.json b/tools/package.json
index e95948841..adf7a0df8 100644
--- a/tools/package.json
+++ b/tools/package.json
@@ -1,6 +1,6 @@
 {
     "name": "tools",
-    "version": "2.31.1",
+    "version": "2.31.2",
     "description": "",
     "main": "clone.js",
     "repository": "https://github.com/dhis2/app-hub",

From 4ff7525e89083dcf6154964762755d38dee65b3d Mon Sep 17 00:00:00 2001
From: Birk Johansson <birk@dhis2.org>
Date: Wed, 16 Nov 2022 13:18:14 +0100
Subject: [PATCH 6/7] fix(apps): fix source-url being cleared when new version
 is uploaded (#697)

---
 .../apps/formatting/convertAppsToApiV1Format.js  | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/server/src/routes/v1/apps/formatting/convertAppsToApiV1Format.js b/server/src/routes/v1/apps/formatting/convertAppsToApiV1Format.js
index 01d408627..7028cfea3 100644
--- a/server/src/routes/v1/apps/formatting/convertAppsToApiV1Format.js
+++ b/server/src/routes/v1/apps/formatting/convertAppsToApiV1Format.js
@@ -4,7 +4,7 @@ const debug = require('debug')(
 const { MediaType } = require('../../../../enums')
 const getServerUrl = require('../../../../utils/getServerUrl')
 
-const convertDbAppViewRowToAppApiV1Object = app => ({
+const convertDbAppViewRowToAppApiV1Object = (app) => ({
     appType: app.type,
 
     status: app.status,
@@ -78,7 +78,7 @@ const convertAll = (apps, request) => {
     debug(`Using serverUrl: ${serverUrl}`)
 
     const formattedApps = {}
-    apps.forEach(app => {
+    apps.forEach((app) => {
         let currentApp = formattedApps[app.app_id]
 
         if (!currentApp) {
@@ -90,7 +90,7 @@ const convertAll = (apps, request) => {
         if (app.media_id !== null) {
             const media = convertAppToV1Media(app, serverUrl)
 
-            if (!currentApp.images.find(img => img.id === media.id)) {
+            if (!currentApp.images.find((img) => img.id === media.id)) {
                 currentApp.images.push(media)
             }
 
@@ -100,9 +100,17 @@ const convertAll = (apps, request) => {
             })
         }
 
+        // some app-version may not have a version set
+        // sourceUrl really should be in 'app'-table, not 'app_version'
+        if (app.source_url && !currentApp.sourceUrl) {
+            currentApp.sourceUrl = app.source_url
+        }
+
         //Prevent duplicate versions
         if (
-            !currentApp.versions.find(version => version.id === app.version_id)
+            !currentApp.versions.find(
+                (version) => version.id === app.version_id
+            )
         ) {
             currentApp.versions.push(convertAppToV1AppVersion(app, serverUrl))
         }

From 6ece4efc327ddc8efe0ee287ce4a7a5012b2ff3c Mon Sep 17 00:00:00 2001
From: "@dhis2-bot" <apps@dhis2.org>
Date: Wed, 16 Nov 2022 12:18:58 +0000
Subject: [PATCH 7/7] chore(release): cut 2.31.3 [skip release]

## [2.31.3](https://github.com/dhis2/app-hub/compare/v2.31.2...v2.31.3) (2022-11-16)

### Bug Fixes

* **apps:** fix source-url being cleared when new version is uploaded ([#697](https://github.com/dhis2/app-hub/issues/697)) ([4ff7525](https://github.com/dhis2/app-hub/commit/4ff7525e89083dcf6154964762755d38dee65b3d))
---
 CHANGELOG.md        | 7 +++++++
 client/package.json | 2 +-
 package.json        | 2 +-
 server/package.json | 2 +-
 tools/package.json  | 2 +-
 5 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 9d0b2f17f..b1a4e5f7c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,10 @@
+## [2.31.3](https://github.com/dhis2/app-hub/compare/v2.31.2...v2.31.3) (2022-11-16)
+
+
+### Bug Fixes
+
+* **apps:** fix source-url being cleared when new version is uploaded ([#697](https://github.com/dhis2/app-hub/issues/697)) ([4ff7525](https://github.com/dhis2/app-hub/commit/4ff7525e89083dcf6154964762755d38dee65b3d))
+
 ## [2.31.2](https://github.com/dhis2/app-hub/compare/v2.31.1...v2.31.2) (2022-11-15)
 
 
diff --git a/client/package.json b/client/package.json
index 004631813..326bb29f7 100644
--- a/client/package.json
+++ b/client/package.json
@@ -1,6 +1,6 @@
 {
     "name": "client",
-    "version": "2.31.2",
+    "version": "2.31.3",
     "description": "The App Hub Client",
     "repository": "https://github.com/dhis2/app-hub",
     "author": "Birk Johansson <birk@dhis2.org>",
diff --git a/package.json b/package.json
index fbc3657ef..4185e428b 100644
--- a/package.json
+++ b/package.json
@@ -36,5 +36,5 @@
     "cypress-cucumber-preprocessor": {
         "nonGlobalStepDefinitions": true
     },
-    "version": "2.31.2"
+    "version": "2.31.3"
 }
diff --git a/server/package.json b/server/package.json
index 89c131a52..11e4e94d6 100644
--- a/server/package.json
+++ b/server/package.json
@@ -1,6 +1,6 @@
 {
     "name": "server",
-    "version": "2.31.2",
+    "version": "2.31.3",
     "description": "The App Hub Server",
     "main": "src/main.js",
     "repository": "https://github.com/dhis2/app-hub",
diff --git a/tools/package.json b/tools/package.json
index adf7a0df8..39d42847c 100644
--- a/tools/package.json
+++ b/tools/package.json
@@ -1,6 +1,6 @@
 {
     "name": "tools",
-    "version": "2.31.2",
+    "version": "2.31.3",
     "description": "",
     "main": "clone.js",
     "repository": "https://github.com/dhis2/app-hub",