Skip to content

fix(editor): move mail editor from ckeditor to tip tap (#170)#6386

Closed
johanah29 wants to merge 5 commits into
mainfrom
issue/tiptap
Closed

fix(editor): move mail editor from ckeditor to tip tap (#170)#6386
johanah29 wants to merge 5 commits into
mainfrom
issue/tiptap

Conversation

@johanah29

Copy link
Copy Markdown
Member

Proposed changes

Testing Instructions

  1. Step-by-step how to test
  2. Environment or config notes

Related issues

  • Closes #ISSUE-NUMBER

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@github-actions github-actions Bot added the filigran team Item from the Filigran team. label Jun 25, 2026
@Filigran-Automation Filigran-Automation changed the title Issue/tiptap feat(frontend): replace CKEditor with a TipTap-based rich text editor Jun 25, 2026
@Filigran-Automation

Copy link
Copy Markdown
Contributor

🤖 [AI-generated]

Hey @johanah29! 👋 Thanks a lot for opening PR #6386 — this is a meaty one (a full rich-text editor swap across 27 files), and we really appreciate the work! 🙏

I just had a quick look and I think the description could be enhanced a little to help reviewers get through a change this size faster. I haven't touched your description — just a gentle, optional nudge:

Area What could help Suggestion
### Proposed changes Currently empty (* bullets) A couple of bullets on the what & why — e.g. replacing CKEditor with a TipTap-based editor, the new rich_text_editor components/extensions, and what motivated the move
### Testing Instructions Placeholder text left as-is A short step-by-step: where the editor is used (e.g. remediation form), what to try (formatting, tables, font size, indent), and anything to watch for on existing content
### Related issues Closes #ISSUE-NUMBER placeholder Link the tracking issue with Closes #<number> — the conventions ask every PR to be linked to an issue 🙂

💡 If helpful, the contribution conventions walk through what to include.

No rush at all — totally fine to fill this in while the draft is open. Thanks again for contributing to OpenAEV! 🚀

@johanah29 johanah29 changed the title feat(frontend): replace CKEditor with a TipTap-based rich text editor fix(mail editor): move mail editor from ckeditor to tip tap (#170) Jun 25, 2026
Signed-off-by: johanah29 <johanah.lekeu@filigran.io>
@johanah29 johanah29 changed the title fix(mail editor): move mail editor from ckeditor to tip tap (#170) fix(editor): move mail editor from ckeditor to tip tap (#170) Jun 25, 2026
Signed-off-by: johanah29 <johanah.lekeu@filigran.io>
@RomuDeuxfois RomuDeuxfois deleted the issue/tiptap branch July 1, 2026 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants