Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testing: Verify that the frontend-lib-special-exams can be added to the openedx-translations #1

Closed
wants to merge 2 commits into from
Closed

Testing: Verify that the frontend-lib-special-exams can be added to the openedx-translations #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
49a1d9a
feat: Add frontend-lib-content-components
OmarIthawi Mar 4, 2023
332fb21
test: frontend-components
OmarIthawi Mar 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
51 changes: 25 additions & 26 deletions .github/workflows/extract-translation-source-files.yml
View file
Open in desktop
Expand Up @@ -5,9 +5,7 @@
name: Extract Translation Source Files

on:
workflow_dispatch: # by request
schedule:
- cron: '0 0 * * *' # every day at midnight
- pull_request
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To make the workflow run on every commit on this pull request so it's possible to test instantly instead of waiting till the next day.


jobs:
setup-branch:
Expand Down Expand Up @@ -40,7 +38,6 @@ jobs:
matrix:
repo:
- course-discovery
- credentials
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove most Python repositories to save GitHubActions time.

runs-on: ubuntu-latest
continue-on-error: true
needs: [setup-branch]
Expand Down Expand Up @@ -129,27 +126,28 @@ jobs:
# repos with errors running extract_translations
# * frontend-template-application
repo:
- frontend-app-account
- frontend-app-authn
- frontend-app-communications
- frontend-app-course-authoring
- frontend-app-discussions
- frontend-app-ecommerce
- frontend-app-enterprise-public-catalog
- frontend-app-gradebook
- frontend-app-learner-dashboard
- frontend-app-learner-record
- frontend-app-learning
- frontend-app-library-authoring
- frontend-app-ora-grading
- frontend-app-payment
- frontend-app-profile
- frontend-app-program-console
- frontend-app-support-tools
- frontend-component-footer
- frontend-component-header
- paragon
- studio-frontend
- frontend-lib-special-exams
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new repository that we're testing the workflow for.

# - frontend-app-account
# - frontend-app-authn
# - frontend-app-communications
# - frontend-app-course-authoring
# - frontend-app-discussions
# - frontend-app-ecommerce
# - frontend-app-enterprise-public-catalog
# - frontend-app-gradebook
# - frontend-app-learner-dashboard
# - frontend-app-learner-record
# - frontend-app-learning
# - frontend-app-library-authoring
# - frontend-app-ora-grading
# - frontend-app-payment
# - frontend-app-profile
# - frontend-app-program-console
# - frontend-app-support-tools
# - frontend-component-footer
# - frontend-component-header
# - paragon
# - studio-frontend
Comment on lines +130 to +150
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove most frontend repositories to save GitHubActions time.

runs-on: ubuntu-latest
continue-on-error: true
needs: [setup-branch, python-translations]
Expand All @@ -164,7 +162,8 @@ jobs:
- name: clone openedx/${{ matrix.repo }}
uses: actions/checkout@v3
with:
repository: openedx/${{ matrix.repo }}
repository: Zeit-Labs/${{ matrix.repo }}
ref: fix-i18n
path: translations/${{ matrix.repo }}

# Sets up node/npm
Expand Down
7 changes: 7 additions & 0 deletions transifex.yml
View file
Open in desktop
Expand Up @@ -150,6 +150,13 @@ git:
source_file: translations/frontend-component-header/src/i18n/transifex_input.json
translation_files_expression: 'translations/frontend-component-header/src/i18n/messages/<lang>.json'

# frontend-lib-content-components
- filter_type: file
file_format: KEYVALUEJSON
source_language: en
source_file: translations/frontend-lib-content-components/src/i18n/transifex_input.json
translation_files_expression: 'translations/frontend-lib-content-components/src/i18n/messages/<lang>.json'
Comment on lines +153 to +158
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will not have affect, until it's actually merged in the Transifex project.


# paragon
- filter_type: file
file_format: KEYVALUEJSON
Expand Down