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

Proof of Concept: Refactor incude library authoring - Don't merge! #731

Closed
wants to merge 6 commits into from

Conversation

jesperhodge
Copy link
Member

@jesperhodge jesperhodge commented Dec 6, 2023

This is an internal proof-of-concept only PR, for now not intended to merge.
Main documentation in our internal confluence (https://2u-internal.atlassian.net/wiki/spaces/TNL/pages/693633214/TNL-11272+Repository+Consolidation).

Important Notes:

  • Unfinished, not ready for merge (only proof of concept)
  • course-import for both course-authoring and library-authoring has been changed to use the same reducer, but I expect this to be a mistake. Better fix
  • library-authoring header has not been added
  • only partially tested: due to local setup problems (unrelated), display and editing of blocks could not be tested

@bradenmacdonald
Copy link
Contributor

I believe this PR can be closed - Axim is starting a project to build an entirely new Library Authoring experience in the Course Authoring MFE, which my team will be building out this summer.

One of our first steps will be implementing support for v2 libraries on the libraries list page: #1033

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.

2 participants