refactor: import authoring filter from content_authoring instead #36109
+6
−6
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When reviewing openedx/openedx-filters#200, I suggested using the subdomain
course_authoring
when it was calledcontent_authoring
. This PR correctly imports the filter implemented in the PR I previously mentioned from thecontent_authoring
subdomain instead of thecourse_authoring
subdomain. This change was made according to the DDD docs for the Open edX ecosystem: https://openedx.atlassian.net/wiki/spaces/AC/pages/663224968/edX+DDD+Bounded+ContextsDepends on openedx/openedx-filters#252
Deadline
Once openedx-filters PR is merged.
Other information
I suggest not backporting this breaking change to the sumac branch to not break existing plugins that are using this filter. It'd be better for them to migrate to this fix once Teak is out.