Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ghassan/community theme.palm #2

Merged
merged 29 commits into from
Jul 10, 2023
Merged

Ghassan/community theme.palm #2

merged 29 commits into from
Jul 10, 2023

Conversation

ghassanmas
Copy link

@ghassanmas ghassanmas commented Jul 10, 2023

This change rebase header to version 4.0.1 which is tested to work with the leanring:

This is tested via:

renovate bot and others added 29 commits March 27, 2023 10:15
…dx#314)

* feat: upgraded to node v18, added .nvmrc and updated workflows

* refactor: upgraded frontend-build & frontend-platform, updated workflows

* refactor: moved platform from devDependencies
* refactor: remove react16 from peer dependency

* build: update lock file

* fix: fix lint issue

* refactor: add 17 support of react in peerDep
gets the intl-imports.js script for FC-0012 OEP-58.
…edx#332)

BREAKING CHANGE: Ending support of @edx/frontend-platform v2 and v3 and now only support v4
ghassanmas added a commit to Abstract-Tech/community-theme-tutor that referenced this pull request Jul 10, 2023
  This change does few thigns:

  1. It upgrade tutor and plugins to plam version >=16
  2. It disbale the old plugin for injectiong post npm install pkgs
  3. And instaed of 2 it uses the new patch that inject per mfe
  4. it create new tutor file plugin named mfe-forks.py that uses
    4.1 the new plam for learning: Abstract-Tech/community-theme-learning/pull/8
    4.2 the new palm 4.0.1 branch for header: Abstract-Tech/community-theme-header/pull/2
@snglth snglth merged commit f2e6846 into master Jul 10, 2023
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.

8 participants