-
Notifications
You must be signed in to change notification settings - Fork 1
Chore: Absorb adapt-authoring-defaultplugins into contentplugin module #91
Copy link
Copy link
Closed
Labels
Description
Chore
- Merge the
adapt-authoring-defaultpluginsmodule intoadapt-authoring-contentplugin
Rationale
As part of an ongoing effort to reduce the number of standalone modules (see adapt-security/adapt-authoring-core#112 for prior art), adapt-authoring-defaultplugins is a strong candidate for absorption.
The module is a single 26-line file (DefaultPluginsModule.js) that hooks into course creation to auto-add default plugins. Its only peer dependency is adapt-authoring-contentplugin — it has no standalone purpose outside of the content plugin lifecycle.
Approach
Following the pattern established in adapt-security/adapt-authoring-core#112:
- Move
DefaultPluginsModulelogic into the contentplugin module as an internal library class (not anAbstractModulesubclass) - Merge
errors.jsondefinitions into the contentplugin error definitions - Merge config schemas into the contentplugin config schema
- Merge documentation and doc plugins (jsdoc-style annotations) into contentplugin
- Move relevant data schemas into contentplugin
- Deprecate the standalone
adapt-authoring-defaultpluginspackage
Reactions are currently unavailable