2022-01-26 Meeting notes

@Adam Stankiewicz

• [inform] Moving to a single documentation website to simplify Netlify/Github integration, and building a theme switcher.

  • Netlify deploy previews on PRs will still work, but rather than deploying 2 sites (one per theme), it’ll deploy the two themes on a single site.

  • https://paragon-openedx.netlify.app/

• Any concerns/feedback?

@TJ Tracy (Deactivated)

TinyMCE in Paragon:

• Ran into issues with unit testing editor, might be a consideration for whether it should go in Paragon

• Unit testing is currently at a very low level

  • Doesn’t cooperate with react-testing-library, Enzyme, etc.

• What are we testing?

  • Mocking TinyMCE?

  • Test the logic in the wrapper

    • Didn’t mock out TinyMCE directly, instead mocked out wrapper

  • TinyMCE officially doesn’t support JSDom

    • Provide a utils to mock the Editor instance in Jest · Issue #91 · tinymce/tinymce-react

  • a11y testing

    • might not need to worry about this, if a11y is baked into the library itself?

  • Continued open questions:

    • Paid plugins like a11ychecker

      • Might need to use CDN version of the library