Skip to content

feat: add frontend-essentials translations#68

Merged
andrey-canon merged 1 commit intoopen-release/teak.nelpfrom
teak-mig/FUTUREX-1283
Oct 9, 2025
Merged

feat: add frontend-essentials translations#68
andrey-canon merged 1 commit intoopen-release/teak.nelpfrom
teak-mig/FUTUREX-1283

Conversation

@andrey-canon
Copy link
Collaborator

@andrey-canon andrey-canon commented Oct 9, 2025

Description

This adds the frontend-essentils package to the pull_translations workflow. migration pr of #45

Issue # 1283

How to test

  1. Checkout this branch
  2. Run the following commands

export PATH="$(pwd)/node_modules/.bin:$PATH"

make OPENEDX_ATLAS_PULL=true ATLAS_OPTIONS="--repository=nelc/futurex-translations --revision=open-release/redwood.master" pull_translations

  1. Verify the translation (Like-dislike and report button, feedback component and profile data component )
image image image

Note

the Arabic name and national id translation is missing the futurex-translations repo

Copy link

@johanseto johanseto left a comment

Choose a reason for hiding this comment

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

LGTM

Working in my teak env

Screenshot from 2025-10-09 13-12-55
Screencast.from.09-10-25.13.11.56.webm

Please add the commands to add translations in the PR body. Step 2 says :
"run the following commands" without comands

@andrey-canon andrey-canon merged commit bed3489 into open-release/teak.nelp Oct 9, 2025
4 checks passed
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp October 9, 2025 18:27 — with GitHub Actions Inactive
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp October 9, 2025 18:27 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants