Skip to content

feat: update @edx/brand dependency to @edx/brand-edx.org - #37236

Merged
UsamaSadiq merged 1 commit into
openedx:masterfrom
ttak-apphelix:ttak-apphelix/revert-branding-alias
Aug 19, 2025
Merged

feat: update @edx/brand dependency to @edx/brand-edx.org#37236
UsamaSadiq merged 1 commit into
openedx:masterfrom
ttak-apphelix:ttak-apphelix/revert-branding-alias

Conversation

@ttak-apphelix

@ttak-apphelix ttak-apphelix commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

Revert PR #37105

Traceback (most recent call last):
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 451, in <module>
    main(prog_name="npm run compile-sass --")
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1442, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1363, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 1226, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/core.py", line 794, in invoke
    return callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/venvs/edxapp/lib/python3.11/site-packages/click/decorators.py", line 34, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 373, in main
    compile_sass_dir(
  File "/edx/app/edxapp/edx-platform/scripts/compile_sass.py", line 252, in compile_sass_dir
    raise Exception(f"Failed to compile {source}: {output_text}")
Exception: Failed to compile lms/static/sass/lms-mobile-rtl.scss: Error: File to import not found or unreadable: brand-edx.org/paragon/fonts
       Parent style sheet: /edx/var/edx-themes/edx-themes/edx-platform/edx.org-next/lms/static/sass/partials/lms/theme/_variables.scss
        on line 4 of ../../../var/edx-themes/edx-themes/edx-platform/edx.org-next/lms/static/sass/partials/lms/theme/_variables.scss
>> @import 'brand-edx.org/paragon/fonts

@UsamaSadiq
UsamaSadiq merged commit 93fb3c4 into openedx:master Aug 19, 2025
49 checks passed
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@robrap

robrap commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

In the future, a revert in the title would help add clarity. Thank you.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

salman2013 pushed a commit to salman2013/edx-platform that referenced this pull request Sep 10, 2025
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.

4 participants