Skip to content

Commit

Permalink
bugfix / fix owner for missing original language (#7591)
Browse files Browse the repository at this point in the history
* fix owner for missing original language

* update tc-source-content-updater

* update deperecated upload-artifact

* update deperecated upload-artifact

* fix paths for upload-artifact

---------

Co-authored-by: PhotoNomad0 <[email protected]>
  • Loading branch information
PhotoNomad0 and PhotoNomad0 authored Sep 16, 2024
1 parent d22e4d4 commit 635b02f
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 32 deletions.
44 changes: 22 additions & 22 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -113,11 +113,11 @@ jobs:
# skip_fetch: true
# skip_checkout: true

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload jsdocs
with:
name: jsdocs
path: ./docs/
path: docs/

build-linux:
name: Build Linux
Expand Down Expand Up @@ -222,13 +222,13 @@ jobs:
run: echo "ARTIFACT_ARM64=$(echo "$APP_NAME-linux-arm64-$APP_VERSION-LITE-$HASH.deb")" >> $GITHUB_ENV

- name: bundle arm64 linux
run: ./node_modules/.bin/gulp release-linux-deb-arm64 --out=../artifacts/arm64/$ARTIFACT_ARM64
run: ./node_modules/.bin/gulp release-linux-deb-arm64 --out=artifacts/arm64/$ARTIFACT_ARM64

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload arm64 artifacts
with:
name: ${{ env.ARTIFACT_ARM64 }}
path: ../artifacts/arm64/
path: artifacts/arm64/

# x64
- name: Set x64 MAX artifact name
Expand All @@ -239,13 +239,13 @@ jobs:
run: echo "ARTIFACT_X64=$(echo "$APP_NAME-linux-x64-$APP_VERSION-LITE-$HASH.deb")" >> $GITHUB_ENV

- name: bundle x64 linux
run: ./node_modules/.bin/gulp release-linux-deb --out=../artifacts/x64/$ARTIFACT_X64
run: ./node_modules/.bin/gulp release-linux-deb --out=artifacts/x64/$ARTIFACT_X64

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload x64 artifacts
with:
name: ${{ env.ARTIFACT_X64 }}
path: ../artifacts/x64/
path: artifacts/x64/

build-windows:
name: Build Windows
Expand Down Expand Up @@ -344,20 +344,20 @@ jobs:

- name: bundle x64 windows
run: |
echo "Generating executable ../artifacts/x64/${{ env.ARTIFACT_X64 }}"
./node_modules/.bin/gulp release-win64 --out=../artifacts/x64/${{ env.ARTIFACT_X64 }}
echo "Generating executable artifacts/x64/${{ env.ARTIFACT_X64 }}"
./node_modules/.bin/gulp release-win64 --out=artifacts/x64/${{ env.ARTIFACT_X64 }}
# - name: list directories
# run: |
# dir .
# dir ../
# dir ../artifacts

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload x64 artifacts
with:
name: ${{ env.ARTIFACT_X64 }}
path: ../artifacts/x64/
path: artifacts/x64/
# x32
- name: Set x32 MAX artifact name
run: echo "ARTIFACT_X32=$(echo "${{ env.APP_NAME }}-win-x32-${{ env.APP_VERSION }}-MAX-${{ env.HASH }}.exe")" >> $GITHUB_ENV
Expand All @@ -370,14 +370,14 @@ jobs:

- name: bundle x32 windows
run: |
echo "Generating executable ../artifacts/x32/${{ env.ARTIFACT_X32 }}"
./node_modules/.bin/gulp release-win32 --out=../artifacts/x32/${{ env.ARTIFACT_X32 }}
echo "Generating executable artifacts/x32/${{ env.ARTIFACT_X32 }}"
./node_modules/.bin/gulp release-win32 --out=artifacts/x32/${{ env.ARTIFACT_X32 }}
- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload x32 artifacts
with:
name: ${{ env.ARTIFACT_X32 }}
path: ../artifacts/x32/
path: artifacts/x32/

build-macos:
name: Build macOS
Expand Down Expand Up @@ -479,13 +479,13 @@ jobs:
run: echo "ARTIFACT_X64=$(echo "$APP_NAME-macos-x64-$APP_VERSION-LITE-$HASH.dmg")" >> $GITHUB_ENV

- name: bundle x64 macOS
run: ./node_modules/.bin/gulp release-macos --out=../artifacts/x64/$ARTIFACT_X64
run: ./node_modules/.bin/gulp release-macos --out=artifacts/x64/$ARTIFACT_X64

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload x64 artifacts
with:
name: ${{ env.ARTIFACT_X64 }}
path: ../artifacts/x64/
path: artifacts/x64/

# universal build
- name: Set universal MAX artifact name
Expand All @@ -496,13 +496,13 @@ jobs:
run: echo "ARTIFACT_UNIV=$(echo "$APP_NAME-macos-universal-$APP_VERSION-LITE-$HASH.dmg")" >> $GITHUB_ENV

- name: bundle universal macOS
run: ./node_modules/.bin/gulp release-macos-universal --out=../artifacts/universal/$ARTIFACT_UNIV
run: ./node_modules/.bin/gulp release-macos-universal --out=artifacts/universal/$ARTIFACT_UNIV

- uses: actions/upload-artifact@v1
- uses: actions/upload-artifact@v4
name: Upload universal artifacts
with:
name: ${{ env.ARTIFACT_UNIV }}
path: ../artifacts/universal/
path: artifacts/universal/

notify:
name: Notify
Expand Down
14 changes: 7 additions & 7 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@
"string-punctuation-tokenizer": "^2.2.0",
"sudo-prompt": "6.2.1",
"tc-electron-env": "0.10.0",
"tc-source-content-updater": "1.4.28",
"tc-source-content-updater": "1.4.29",
"tc-strings": "0.1.7",
"tc-tool": "4.1.0",
"tc-ui-toolkit": "6.2.9",
Expand Down
5 changes: 3 additions & 2 deletions src/js/actions/MyProjects/ProjectLoadingActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -419,13 +419,14 @@ export const showInvalidOrigLangVersionError = (manifest) => (dispatch, getState
const upgradeText = translate('buttons.update_version');
const bookId = manifest?.project?.id || '';
const { bibleId: origLangBibleId, languageId: origLangId } = BibleHelpers.getOrigLangforBook(bookId);
const origLangOwnerForWA = manifest?.toolsSelectedOwners?.wordAlignment;
let origLangOwnerForWA = manifest?.toolsSelectedOwners?.wordAlignment;
origLangOwnerForWA = getOriginalLangOwner(origLangOwnerForWA || DEFAULT_ORIG_LANG_OWNER);
const origLangEditVersionForWA = manifest?.tc_orig_lang_check_version_wordAlignment;
const missingOLResource = {
languageId: origLangId,
resourceId: origLangBibleId,
version: origLangEditVersionForWA || 'master',
owner: origLangOwnerForWA || DEFAULT_ORIG_LANG_OWNER,
owner: origLangOwnerForWA,
};

console.log(`showInvalidOrigLangVersionError() - get missing OL resource: ${JSON.stringify(missingOLResource)}`);
Expand Down

0 comments on commit 635b02f

Please sign in to comment.