Skip to content
Snippets Groups Projects

CKEditor5 Blog PoC

Merged Ben requested to merge feat/ckeditor-blogs-2333 into epic/ckeditor5-blogs

Summary

Closes #2333 (closed)

Depends on engine!448 (merged) This is a PoC for testing and evaluation.

We are going to need to fork the balloon editor and upload the package via NPM, so that we can make a lightly customized build where we enable the Alignment plugin. Because we are going the custom-build route, there is also no image upload, as the code for the UploadAdapter will likely change when separating from the Balloon build.

Steps to test

  1. Make a new blog.
  2. Edit it. The url should still be /blog/edit/:guid
  3. Old editor should still work fine.
  4. Change the URL to blog-v2 e.g. /blog-v2/edit/:guid
  5. Play with new editor, remember alignment and upload are going to require additional work.
  6. Save.
  7. Hit edit again, it should link to the /blog-v2/edit/:guid again as the editor_version variable has changed

Estimated Regression Scope

As this is not to go live; there isn't a huge scope for regression. If this is tested on Staging or in another environment, any regressions would be in former blogs, as the v2-blogs route isn't known to users.

Edited by Ben

Merge request reports

Checking pipeline status.

Merged by Xander MillerXander Miller 5 years ago (Feb 26, 2020 12:11am UTC)

Loading

Pipeline #121033694 passed with warnings

Pipeline passed with warnings for 0719a794 on epic/ckeditor5-blogs

4 environments impacted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Mark Harding
  • Ben added 1 commit

    added 1 commit

    Compare with previous version

  • Ben added 3 commits

    added 3 commits

    Compare with previous version

  • Ben changed the description

    changed the description

  • Ben added 1 commit

    added 1 commit

    • 9048abd6 - updated to add custom editor_version attribute and digest it

    Compare with previous version

  • Ben unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Ben changed the description

    changed the description

  • Ben added 1 deleted label and removed StatusRequires Changes label

    added 1 deleted label and removed StatusRequires Changes label

  • Ben added 1 commit

    added 1 commit

    • 57e53512 - Tidying and removing legacy tests

    Compare with previous version

  • Ben added 34 commits

    added 34 commits

    • 57e53512...9f5a7a2b - 33 commits from branch master
    • fa17ae44 - Merge branch 'master' of gitlab.com:minds/front into feat/ckeditor-blogs-2333

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading