Skip to content
This repository has been archived by the owner on Oct 18, 2024. It is now read-only.

Update frontend dependency (and bring along new exercise plugin) #298

Merged
merged 10 commits into from
Nov 20, 2023

Conversation

LarsTheGlidingSquirrel
Copy link
Member

@LarsTheGlidingSquirrel LarsTheGlidingSquirrel commented Nov 17, 2023

Sub-tasks

  • Get the editor to open again when running yarn dev
  • Fix issue with saving content
  • Refactor plugin edusharingAsset to use new structure (*Editor & *StaticRenderer)
  • Fix yarn lint errors introduced by new project configuration in frontend
  • Refactor plugin serloInjection to use new structure (*Editor & *StaticRenderer)
  • Fix broken geogebra static renderer view -> Gave up on it and deactivated geogebra plugin for now
  • Test editor for new bugs
  • yarn docker:build still works
  • All tests pass

Breaking changes

  • Plugins ScMcExercise and InputExercise can no longer be created on their own but they can be part of the new Exercise plugin. Old content might still be shown correctly but I would advise deleting all old content in the testing environment.
  • Deactivated geogebra plugin because its static renderer view is broken. Tried to fix it but did not find a solution. Postponed.

Copy link

vercel bot commented Nov 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ece-as-a-service ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2023 5:15pm

@LarsTheGlidingSquirrel LarsTheGlidingSquirrel marked this pull request as ready for review November 20, 2023 08:03
@LarsTheGlidingSquirrel LarsTheGlidingSquirrel merged commit da28ef4 into main Nov 20, 2023
9 checks passed
@LarsTheGlidingSquirrel LarsTheGlidingSquirrel deleted the update-frontend-dependency branch November 20, 2023 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant