Merge branch 'develop_tchap' into 485-when-i-try-to-join-a-room-witho…

…ut-rights-to-it-error-message-is-in-english

.github/workflows/build.yml

@@ -1,5 +1,4 @@
 #Build the application
-#tchap modification : use "./scripts/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
 name: Build
 on:
     pull_request: {}
@@ -23,7 +22,7 @@ jobs:
                   node-version: 16
 
             - name: Install Dependencies
-              run: "./scripts/install-yarn-linked-repositories.sh"
+              run: "yarn install"
 
             - name: Build
               run: "yarn build"

.github/workflows/create-release.yml

@@ -1,4 +1,4 @@
-name: Create tag and release for tchap-web
+name: Build archives and create release for tchap-web on tag "tchap-"
 
 on:
     push:
@@ -11,6 +11,7 @@ jobs:
         permissions:
             contents: write
         steps:
+            #Build artifacts for dev, preprod and prod
             - uses: actions/checkout@v3
             - name: Get Node Version
               id: node_version
@@ -19,24 +20,28 @@ jobs:
               uses: actions/setup-node@v3
               with:
                   node-version: ${{ steps.node_version.outputs.node_version }}
+            - name: Install Dependencies
+              run: "yarn install"
             - name: Get Tchap Web Version
               id: version
               run: echo ::set-output name=version::$(node -e 'console.log(require("./package.json").version)')
             - name: Build the archive ${{ steps.version.outputs.version }} for dev
-              run: scripts/package-github.sh
+              run: scripts/tchap/package-tchap.sh
               env:
                   CONFIG: dev
             - name: Build the archive ${{ steps.version.outputs.version }} for preprod
-              run: scripts/package-github.sh
+              run: scripts/tchap/package-tchap.sh
               env:
                   CONFIG: preprod
             - name: Build the archive ${{ steps.version.outputs.version }} for prod
-              run: scripts/package-github.sh
+              run: scripts/tchap/package-tchap.sh
               env:
                   CONFIG: prod
             - name: Get current date
               id: date
               run: echo "::set-output name=date::$(date +'%Y%m%d')"
+
+            #create release
             - name: Create tchap-web release
               uses: ncipollo/release-action@v1
               with:

.github/workflows/cypress.yaml

@@ -1,6 +1,6 @@
 # Run the tests
 # Todo : reuse a build from another action, instead of rebuilding for nothing.
-# tchap modification : use "./scripts/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
+# tchap modification : use "./scripts/tchap/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
 name: Cypress End to End Tests
 on:
     pull_request: {}
@@ -28,7 +28,7 @@ jobs:
                   node-version: 16
 
             - name: Install Dependencies
-              run: "./scripts/install-yarn-linked-repositories.sh"
+              run: "./scripts/tchap/install-yarn-linked-repositories.sh"
 
             - name: Run Cypress tests
               uses: cypress-io/[email protected]

.github/workflows/static_analysis.yaml

@@ -1,5 +1,4 @@
 #Run a static analysis
-#tchap modification : use "./scripts/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
 name: Static Analysis
 on:
     pull_request: {}
@@ -27,7 +26,7 @@ jobs:
     #          node-version: 16
     #
     #      - name: Install Dependencies
-    #        run: "./scripts/install-yarn-linked-repositories.sh"
+    #        run: "./scripts/tchap/install-yarn-linked-repositories.sh"
     #
     #      - name: Typecheck
     #        run: "yarn run lint:types"
@@ -68,24 +67,7 @@ jobs:
 
             # Needs branch matching as it inherits .stylelintrc.js from matrix-react-sdk
             - name: Install Dependencies
-              run: "./scripts/install-yarn-linked-repositories.sh"
+              run: "yarn install --pure-lockfile"
 
             - name: Run Linter
               run: "yarn run lint:style"
-# tchap :  analyse:unused-exports does not work well with tchap customisations, it raises exception for "default export" not used
-#analyse_dead_code:
-#   name: "Analyse Dead Code"
-#   runs-on: ubuntu-latest
-#   steps:
-#     - uses: actions/checkout@v2
-
-#     - uses: actions/setup-node@v3
-#       with:
-#         cache: 'yarn'
-#         node-version: 16
-
-#     - name: Install Deps
-#       run: "scripts/install-yarn-linked-repositories.sh"
-
-#     - name: Dead Code Analysis
-#       run: "yarn run analyse:unused-exports"

.github/workflows/tests.yaml

@@ -1,5 +1,5 @@
 #Run the tests
-#tchap modification : use "./scripts/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
+#tchap modification : use "./scripts/tchap/install-yarn-linked-repositories.sh" instead of "./scripts/layered.sh"
 name: Tests
 on:
     pull_request: {}
@@ -26,7 +26,7 @@ jobs:
                   node-version: 16
 
             - name: Install Dependencies
-              run: "./scripts/install-yarn-linked-repositories.sh"
+              run: "./scripts/tchap/install-yarn-linked-repositories.sh"
 
             - name: Run tests with coverage
               run: "yarn coverage --ci"

.gitignore

@@ -28,7 +28,7 @@ electron/pub
 /matrix-react-sdk
 /matrix-js-sdk
 /matrix-analytics-events
-# yarn links dependencies generated with /scripts/install-yarn-linked-repositories.sh
+# yarn links dependencies generated with /scripts/tchap/install-yarn-linked-repositories.sh
 /yarn-linked-dependencies
 
 # ignore folder for patch merging conflicts

.prettierignore

@@ -28,3 +28,4 @@ src/vector/modernizr.js
 
 #remove tchap files from prettier
 patches/patches.json
+yarn-linked-dependencies/

README.md

@@ -323,7 +323,7 @@ yarn start
 Alternatively, you can skip the above commands and only run the command below. The only difference is that both 'matrix-react-sdk and 'matrix-js-sdk' will be cloned inside the root project folder.
 
 ```bash
-./scripts/install-yarn-linked-repositories.sh
+./scripts/tchap/install-yarn-linked-repositories.sh
 yarn install
 yarn start
 ```

package.json

@@ -27,6 +27,9 @@
         "contribute.json"
     ],
     "style": "bundle.css",
+    "scripts_comments": {
+        "build:jitsi": "tchap don't need that, deactivate?"
+    },
     "scripts": {
         "i18n": "matrix-gen-i18n",
         "prunei18n": "matrix-prune-i18n",
@@ -43,8 +46,8 @@
         "build:module_system": "tsc --project ./tsconfig.module_system.json && node ./lib/module_system/scripts/install.js",
         "build:github": "yarn build:genfiles && yarn build:bundle",
         "dist": "scripts/package.sh",
-        "scalingo-postbuild": "scripts/package-scalingo-prod.sh",
-        "start": "export VERSION=$(scripts/get-version-from-package-json.sh) && yarn build:module_system && concurrently --kill-others-on-fail --prefix \"{time} [{name}]\" -n res,element-js \"yarn start:res\" \"yarn start:js\"",
+        "scalingo-postbuild": "scripts/tchap/package-tchap.sh",
+        "start": "export VERSION=$(scripts/tchap/get-version-from-package-json.sh) && yarn build:module_system && concurrently --kill-others-on-fail --prefix \"{time} [{name}]\" -n res,element-js \"yarn start:res\" \"yarn start:js\"",
         "start:https": "concurrently --kill-others-on-fail --prefix \"{time} [{name}]\" -n res,element-js \"yarn start:res\" \"yarn start:js --https\"",
         "start:res": "yarn build:jitsi && node scripts/copy-res.js -w",
         "start:js": "webpack-dev-server --output-filename=bundles/_dev_/[name].js --output-chunk-filename=bundles/_dev_/[name].js -w --mode development --disable-host-check --hot",
@@ -64,10 +67,10 @@
         "test:cypress:open": "cypress open",
         "coverage": "yarn test --coverage",
         "analyse:unused-exports": "node ./scripts/analyse_unused_exports.js",
-        "postinstall": "./scripts/apply_patches.sh",
+        "postinstall": "./scripts/tchap/apply_patches.sh",
         "patch-package": "patch-package",
         "patches-reapply": "rm -rf node_modules/matrix-react-sdk; rm -rf node_modules/matrix-js-sdk; yarn install --force",
-        "patch-make": "node scripts/makePatch.ts"
+        "patch-make": "node scripts/tchap/makePatch.ts"
     },
     "dependencies": {
         "@matrix-org/olm": "https://gitlab.matrix.org/api/v4/projects/27/packages/npm/@matrix-org/olm/-/@matrix-org/olm-3.2.12.tgz",

res/themes/tchap-common/css/_tchap_custom.pcss

@@ -19,7 +19,7 @@ div.mx_SettingsTab.mx_SecurityUserSettingsTab > div:nth-child(4) > div:nth-child
 }
 
 /* change link color and decoration in login error when email is not autorized */
-.mx_Login_error.mx_Login_error_link{
+.mx_Login_error.mx_Login_error_link {
     color: $alert important!;
     text-decoration: underline important!;
-}
+}

scripts/apply_patches.sh → scripts/tchap/apply_patches.sh

@@ -3,7 +3,7 @@
 # Script to apply the patches in patches directory, using patch-package.
 set -e
 
-node scripts/checkPatchFile.js
+node scripts/tchap/checkPatchFile.js
 
 for d in patches/*/ ; do
     echo "Patching $d..."

scripts/checkPatchFile.js → scripts/tchap/checkPatchFile.js

@@ -69,7 +69,7 @@ const getPatchDirs = () => {
         .map((dir) => dir.name);
 };
 
-const patchFile = require("../patches/patches.json");
+const patchFile = require("../../patches/patches.json");
 const patchDirs = getPatchDirs();
 checkForDuplicateFiles(patchFile);
 checkForMissingPatchDirs(patchFile, patchDirs);

scripts/install-yarn-linked-repositories.sh → scripts/tchap/install-yarn-linked-repositories.sh

@@ -32,7 +32,7 @@ mkdir -p yarn-linked-dependencies
 cd yarn-linked-dependencies
 
 # Set up the js-sdk first
-../scripts/fetchdep.with.version.sh matrix-org matrix-js-sdk $MATRIX_JS_SDK_VERSION
+../scripts/tchap/fetchdep.with.version.sh matrix-org matrix-js-sdk $MATRIX_JS_SDK_VERSION
 pushd matrix-js-sdk
 yarn unlink # :TCHAP: for local build, undo previous links if present.
 yarn link
@@ -50,7 +50,7 @@ popd
 #popd
 
 # Now set up the react-sdk
-../scripts/fetchdep.with.version.sh matrix-org matrix-react-sdk $MATRIX_REACT_SDK_VERSION
+../scripts/tchap/fetchdep.with.version.sh matrix-org matrix-react-sdk $MATRIX_REACT_SDK_VERSION
 pushd matrix-react-sdk
 yarn unlink # :TCHAP: for local build, undo previous links if present.
 yarn link

scripts/makePatch.ts → scripts/tchap/makePatch.ts

@@ -1,4 +1,4 @@
-const patchFile = require("../patches/patches.json");
+const patchFile = require("../../patches/patches.json");
 
 if (process.argv.length !== 3) {
     console.log("\nMissing patch name.");

scripts/package-github.sh → scripts/tchap/package-github.sh

@@ -1,12 +1,14 @@
 #!/bin/bash
 
+#DEPRECRATED, see package-tchap.sh
+
 # Tchap: File copied from scripts/package.json.
 
 set -e
 
 today=$(date +%Y%m%d)
 
-version=$(./scripts/get-version-from-package-json.sh)
+version=$(./scripts/tchap/get-version-from-package-json.sh)
 export VERSION=$version
 echo "VERSION is set to $VERSION"
 

scripts/package-scalingo-prod.sh → scripts/tchap/package-scalingo-prod.sh

@@ -1,12 +1,15 @@
 #!/bin/bash
 
+#DEPRECRATED, see package-tchap.sh
+
 # Tchap: File copied from scripts/package.json.
+# use package-github instead
 
 set -e
 
 today=$(date +%Y%m%d)
 
-version=$(./scripts/get-version-from-package-json.sh)
+version=$(./scripts/tchap/get-version-from-package-json.sh)
 export VERSION=$version
 echo "VERSION is set to $VERSION"
 
@@ -33,8 +36,8 @@ fi
 
 # Copy the files in /dist, ready to be served.
 cp -r tchap-$version/* dist/
-# Also make a tar file. Useful for releases.
-tar chvzf tchap-$version-$CONFIG-$today.tar.gz dist
+# Also make a tar file. Useful for releases. EDIT: not needed
+#tar chvzf tchap-$version-$CONFIG-$today.tar.gz dist
 rm -r tchap-$version
 
 echo

scripts/tchap/package-tchap.sh

@@ -0,0 +1,43 @@
+#!/bin/bash
+
+# Tchap: File copied from scripts/package.js
+# This file is meant to be used in scalingo and github actions
+
+set -e
+
+today=$(date +%Y%m%d)
+
+version=$(./scripts/tchap/get-version-from-package-json.sh)
+export VERSION=$version
+echo "VERSION is set to $VERSION"
+
+if [[ -n "$CONFIG" ]]; then
+  echo "CONFIG=$CONFIG"
+  cp "config.$CONFIG.json" config.json
+  echo "Using config.$CONFIG.json"
+else
+  echo "No config specified, using config.json."
+fi
+
+yarn clean
+yarn build
+
+mkdir -p dist
+cp -r webapp tchap-$version
+
+# if $version looks like semver with leading v, strip it before writing to file
+if [[ ${version} =~ ^v[[:digit:]]+\.[[:digit:]]+\.[[:digit:]]+(-.+)?$ ]]; then
+    echo ${version:1} > tchap-$version/version
+else
+    echo ${version} > tchap-$version/version
+fi
+
+# Copy the files in /dist, ready to be served.
+cp -r tchap-$version/* dist/
+
+# Also make a tar file. Useful for releases.
+#tar chvzf tchap-$version-$CONFIG-$today.tar.gz dist
+rm -r tchap-$version
+
+echo
+echo "Packaged tchap-$version-$CONFIG"

src/util/TchapUrls.ts

@@ -1,8 +1,4 @@
-
-
 export default class TchapUrls {
-
     //url to request the opening of a new domain on Tchap
     public static requestDomainUrl = "https://www.demarches-simplifiees.fr/commencer/utiliser-tchap";
-
-}
+}