feat: upload artefact for QA

oxen-io · Sep 12, 2024 · da975a1 · da975a1
1 parent 35a02a4
commit da975a1
Showing 1 changed file with 55 additions and 40 deletions.
diff --git a/.github/workflows/check_for_crowdin_updates.yml b/.github/workflows/check_for_crowdin_updates.yml
@@ -25,7 +25,7 @@ jobs:
         with:
           repository: 'oxen-io/session-desktop'
           path: 'desktop'
-          ref: 'unstable'
+          ref: 'standardised_strings_qa_2'
       - name: Checkout iOS
         uses: actions/checkout@v4
         with:
@@ -64,54 +64,59 @@ jobs:
             "${{ github.workspace }}/raw_translations" \
             "${{ github.workspace }}/desktop/_locales" \
             "${{ github.workspace }}/desktop/ts/localization/constants.ts"
+      - name: Prepare QA strings
+        run: |
+          cd ${{ github.workspace }}/desktop/
+          python ./tools/localization/generateLocales.py --generate-types --print-problems --print-problem-strings
+          cd -
       - name: Prepare iOS Strings
         run: |
           python "${{ github.workspace }}/scripts/crowdin/generate_ios_strings.py" \
             "${{ github.workspace }}/raw_translations" \
             "${{ github.workspace }}/ios/Session/Meta" \
             "${{ github.workspace }}/ios/SessionUtilitiesKit/General/Constants.swift"
-      - name: Create Android Pull Request
-        uses: peter-evans/create-pull-request@v6
-        with:
-          path: 'android'
-          token: ${{ secrets.CROWDIN_PR_TOKEN }}
-          title: "[Automated] Update translations from Crowdin"
-          body: |
-            [Automated]
-            This PR includes the latest translations from Crowdin
+      # - name: Create Android Pull Request
+      #   uses: peter-evans/create-pull-request@v6
+      #   with:
+      #     path: 'android'
+      #     token: ${{ secrets.CROWDIN_PR_TOKEN }}
+      #     title: "[Automated] Update translations from Crowdin"
+      #     body: |
+      #       [Automated]
+      #       This PR includes the latest translations from Crowdin
 
-            Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
-          branch: feature/update-crowdin-translations
-          commit-message: "[Automated] Update translations from Crowdin"
-          delete-branch: true
-      - name: Create Desktop Pull Request
-        uses: peter-evans/create-pull-request@v6
-        with:
-          path: 'desktop'
-          token: ${{ secrets.CROWDIN_PR_TOKEN }}
-          title: "[Automated] Update translations from Crowdin"
-          body: |
-            [Automated]
-            This PR includes the latest translations from Crowdin
+      #       Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
+      #     branch: feature/update-crowdin-translations
+      #     commit-message: "[Automated] Update translations from Crowdin"
+      #     delete-branch: true
+      # - name: Create Desktop Pull Request
+      #   uses: peter-evans/create-pull-request@v6
+      #   with:
+      #     path: 'desktop'
+      #     token: ${{ secrets.CROWDIN_PR_TOKEN }}
+      #     title: "[Automated] Update translations from Crowdin"
+      #     body: |
+      #       [Automated]
+      #       This PR includes the latest translations from Crowdin
 
-            Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
-          branch: feature/update-crowdin-translations
-          commit-message: "[Automated] Update translations from Crowdin"
-          delete-branch: true
-      - name: Create iOS Pull Request
-        uses: peter-evans/create-pull-request@v6
-        with:
-          path: 'ios'
-          token: ${{ secrets.CROWDIN_PR_TOKEN }}
-          title: "[Automated] Update translations from Crowdin"
-          body: |
-            [Automated]
-            This PR includes the latest translations from Crowdin
+      #       Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
+      #     branch: feature/update-crowdin-translations
+      #     commit-message: "[Automated] Update translations from Crowdin"
+      #     delete-branch: true
+      # - name: Create iOS Pull Request
+      #   uses: peter-evans/create-pull-request@v6
+      #   with:
+      #     path: 'ios'
+      #     token: ${{ secrets.CROWDIN_PR_TOKEN }}
+      #     title: "[Automated] Update translations from Crowdin"
+      #     body: |
+      #       [Automated]
+      #       This PR includes the latest translations from Crowdin
 
-            Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
-          branch: feature/update-crowdin-translations
-          commit-message: "[Automated] Update translations from Crowdin"
-          delete-branch: true
+      #       Session uses the community-driven translation platform Crowdin for localization, anyone can contribute at https://getsession.org/translate
+      #     branch: feature/update-crowdin-translations
+      #     commit-message: "[Automated] Update translations from Crowdin"
+      #     delete-branch: true
       - name: Upload Android artifacts
         uses: actions/upload-artifact@v4
         with:
@@ -132,6 +137,16 @@ jobs:
           overwrite: true
           if-no-files-found: warn
           retention-days: 7
+      - name: Upload QA artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: session-qa-artifact
+          path: |
+            ${{ github.workspace }}/desktop/ts/localization/locales.ts
+            ${{ github.workspace }}/desktop/ts/localization/constants.ts
+          overwrite: true
+          if-no-files-found: warn
+          retention-days: 7
       - name: Upload iOS artifacts
         uses: actions/upload-artifact@v4
         with: