Skip to content

feat: add set custom primary colors method in initialize flow - #17

Merged
andrey-canon merged 3 commits into
open-release/verawood.nelpfrom
and/FUTUREX-1454
Aug 13, 2026
Merged

feat: add set custom primary colors method in initialize flow#17
andrey-canon merged 3 commits into
open-release/verawood.nelpfrom
and/FUTUREX-1454

Conversation

@andrey-canon

@andrey-canon andrey-canon commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Description:

This allows to set the primary colors based on the setting CUSTOM_PRIMARY_COLORS.

Issue # 1454
This is a migration pr of #11

How to test

  1. You need a functional verawood MFE in your local machine. This instructions consider frontend-app-learning branch (open-release/verawood.nelp)
  2. Add the module.config.js file into the frontend-app-learning folder, The frontend-platform project must be accessible from the mfe tutor container
module.exports = {
    localModules: [
        { moduleName: '@edx/frontend-platform', dir: '../frontend-platform', dist: 'src' },
    ],
};
  1. Checkout this branch
  2. Ad the following setting to the MFE_CONFIG(env/apss/openedx/settings/lms/development.py)
MFE_CONFIG["CUSTOM_PRIMARY_COLORS"] = {
    "pgn-color-primary-base": "#00AA00",
}

Expected result

image

@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 local env.

image

@andrey-canon
andrey-canon changed the base branch from and/FUTUREX-1453 to open-release/verawood.nelp August 13, 2026 18:04
@andrey-canon
andrey-canon merged commit 8a16c13 into open-release/verawood.nelp Aug 13, 2026
3 checks passed
@andrey-canon
andrey-canon deployed to open-release/verawood.nelp August 13, 2026 18:05 — with GitHub Actions Active
@andrey-canon
andrey-canon deployed to open-release/verawood.nelp August 13, 2026 18:05 — with GitHub Actions Active
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