Skip to content

[17.0][MIG] web_editor_class_selector: Migration to version 17.0#2991

Merged
OCA-git-bot merged 5 commits intoOCA:17.0from
Tecnativa:17.0-mig-web_editor_class_selector
Nov 20, 2024
Merged

[17.0][MIG] web_editor_class_selector: Migration to version 17.0#2991
OCA-git-bot merged 5 commits intoOCA:17.0from
Tecnativa:17.0-mig-web_editor_class_selector

Conversation

@carlos-lopez-tecnativa
Copy link
Copy Markdown
Contributor

TT51657
@Tecnativa @pedrobaeza @chienandalu could you please review this

… editor.

This module allows users to create custom CSS classes, which can then be selected and applied directly in the HTML editor.
…s not present

Go to the website
Open the editor
Add text and select it, a traceback is shown
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on base runboat installing the module, and working. There's any module in the repo that is breaking the styles popup toolbar, so it's not possible to be tested on regular runboat.

Comment thread web_editor_class_selector/README.rst
@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot migration web_editor_class_selector

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Nov 19, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 19, 2024
33 tasks
Copy link
Copy Markdown
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test and code review.

Non really blocking comment:

Comment thread web_editor_class_selector/static/src/js/backend/html_field.esm.js Outdated
@carlos-lopez-tecnativa carlos-lopez-tecnativa force-pushed the 17.0-mig-web_editor_class_selector branch from 063c799 to 1a2cab1 Compare November 20, 2024 13:46
@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-2991-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f107681 into OCA:17.0 Nov 20, 2024
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 955ab1d. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza pedrobaeza deleted the 17.0-mig-web_editor_class_selector branch November 20, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants