Skip to content

Sync master to design tokens - #2003

Merged
brian-smith-tcril merged 8 commits into
openedx:master-design-tokensfrom
brian-smith-tcril:sync-master-to-design-tokens
May 20, 2025
Merged

Sync master to design tokens#2003
brian-smith-tcril merged 8 commits into
openedx:master-design-tokensfrom
brian-smith-tcril:sync-master-to-design-tokens

Conversation

@brian-smith-tcril

Copy link
Copy Markdown
Contributor

LAND AS MERGE COMMIT

bradenmacdonald and others added 8 commits May 12, 2025 15:24
Add the additional source `plugins` folder on the `fedx-scripts` extract translations.
This can be configured because the frontend-build v14.5.0 now supports additional source folders using the `--include` extra argument.
* fix: adding messages for i18n issues related to aria labels

* fix: replacing ternary operator

* refactor: injecting intl object to prevent extra modifications on license details component

* test: fixing failing snapshot test

* chore: best descriptions to aria label and removing aria label not needed

* test: snapshot update and fixing ut

* chore: best descriptions to aria labels
- add never show assessment result message in subsection card
…x#1469)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@brian-smith-tcril brian-smith-tcril added the create-sandbox open-craft-grove should create a sandbox environment from this PR label May 19, 2025
@codecov

codecov Bot commented May 19, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (master-design-tokens@3c52a28). Learn more about missing BASE report.

Additional details and impacted files
@@                   Coverage Diff                   @@
##             master-design-tokens    #2003   +/-   ##
=======================================================
  Coverage                        ?   93.89%           
=======================================================
  Files                           ?     1154           
  Lines                           ?    24061           
  Branches                        ?     5200           
=======================================================
  Hits                            ?    22592           
  Misses                          ?     1401           
  Partials                        ?       68           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@brian-smith-tcril brian-smith-tcril removed the create-sandbox open-craft-grove should create a sandbox environment from this PR label May 19, 2025
@brian-smith-tcril
brian-smith-tcril merged commit 07a9e7b into openedx:master-design-tokens May 20, 2025
@bradenmacdonald

Copy link
Copy Markdown
Contributor

@brian-smith-tcril @adamstankiewicz What's the plan for landing v23 / design tokens in this MFE (merging to master)? Is there a list of blockers somewhere?

@brian-smith-tcril

Copy link
Copy Markdown
Contributor Author

@bradenmacdonald the plan is to land all of the master-design-tokens branches of MFEs on June 18th. The date was chosen to ensure orgs that deploy from master have adequate time to prepare for the breaking change. Adam has detailed what is needed for 2U to be ready for this to land on edx.org in his comment here openedx/brand-openedx#23 (comment)

luisfelipec95 pushed a commit to eduNEXT/frontend-app-course-authoring that referenced this pull request Jul 4, 2025
…-design-tokens

Sync master to design tokens
luisfelipec95 pushed a commit to eduNEXT/frontend-app-course-authoring that referenced this pull request Jul 4, 2025
…-design-tokens

Sync master to design tokens
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants