Skip to content

feat: add frontend-essentials translations - #68

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

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

Conversation

@andrey-canon

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

Copy link
Copy Markdown
Collaborator

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

@johanseto johanseto left a comment

Copy link
Copy Markdown

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