[SE-3381] Allows adding new tinymce plugins through platform configuration

[nizarmah]

This makes it possible to install new tinymce plugins through the environment settings.

This PR is responsible for adding the tinymce plugins to the plugins directory and rebuilding the assets.

JIRA tickets: SE-3381, SE-3247

Dependencies:

• [SE-3401] Updates JakePackage.zip to make JS minification consistent across different machines openedx-platform#275

Upstream Pull Request:

• https://github.com/edx/configuration/pull/6179

Sandbox URL:

• LMS: https://configurable-tinymce-plugins.stage.opencraft.hosting/
• Studio: https://studio.configurable-tinymce-plugins.stage.opencraft.hosting/

Testing instructions:

1. Login to Studio using staff@example.com/edx
2. Create a new Unit with an HTML Text
3. Make sure that the ADSK Link shows in the toolbar
4. Create a link and make sure you can preview it, and see it on the LMS

Reviewers

• @pkulkark
• @bradenmacdonald

Settings

EDXAPP_CMS_ENV_EXTRA:
  JS_ENV_EXTRA_CONFIG:
    TINYMCE_ADDITIONAL_PLUGINS:
      adsklink:
        linktypes:
        - download
        - offer
        filetypes:
        - PDF
        - zip
        - Video
        - Design
        orientations:
        - Vertical
        - Horizontal
        styles:
        - Primary
        - Normal
        - Secondary

TINYMCE_ADDITIONAL_PLUGINS_LIST:
- repo: https://gitlab.com/nizarmah/tinymce-adsk-plugin
  name: adsklink
  plugin_path: "/adsklink"

edx_ansible_source_repo: "https://github.com/open-craft/configuration.git"
configuration_version: "nizar/tinymce_plugins_role_backport"
edx_platform_commit: "nizar/tinymce-easy-plugin-modification-backport"

---

Make sure that the following steps are done before merging:

• A DevOps team member has approved the PR if it is code shared across multiple services and you don't own all of the services.
• Are you adding any new default values that need to be overridden when this change goes live? If so:
  • Update the appropriate internal repo (be sure to update for all our environments)
  • If you are updating a secure value rather than an internal one, file a DEVOPS ticket with details.
  • Add an entry to the CHANGELOG.
• If you are making a complicated change, have you performed the proper testing specified on the Ops Ansible Testing Checklist? Adding a new variable does not require the full list (although testing on a sandbox is a great idea to ensure it links with your downstream code changes).
• Think about how this change will affect Open edX operators. Have you updated the wiki page for the next Open edX release?

[pkulkark]

@nizarmah It looks like the styling is not getting applied inside the html editor or in LMS of the sandbox. But it's loading correctly in Studio. This comment might help understand what css file gets loaded when. Maybe you missed a step somewhere? Or missed applying the theme?

[nizarmah]

@pkulkark I've been investigating this for a while now. I just realized that there's a separate theme related change for this tinymce plugin specifically. Here's the commit I am talking about.

I'm currently trying to make the platform include the stylesheets 👀
Let's hope this works out and people don't need to add the styles manually 🤞

[nizarmah]

@pkulkark small note, we might want to continue this conversation on this pull request instead, since it is more related to the edX platform than the configuration.

[pkulkark]

@nizarmah The changes LGTM 👍

• I tested this: verified the feature works as expected in the sandbox deployed using this config.
• I read through the code

[nizarmah]

👍🏼

I'll be merging the changes.

• Changes are being reviewed upstream
• Addressed first set of comments on upstream pull request
• Tested the new changes on the sandbox instance
• Pull request was already reviewed by another member

If any changes are requested upstream, we can revert, and re-apply the changes.