Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Use editor debug component instead of duplicated code (see GN-3152) #3

Merged
merged 6 commits into from Feb 2, 2022

Conversation

benjay10
Copy link
Contributor

This plugin now uses the debug component from the ember-rdfa-editor instead of duplicating code. As of now this depends on release 0.50.0-beta.6 of the editor.
Was tested, but please, test the build and its dependencies as a double check.

“[BenJay10]” added 4 commits January 26, 2022 23:18
Removed the dummy app code that is now a duplicate of the rdfa-editor
debug component (see issue GN-3152) and used that instead.
Forgot to remove the unused code in previous commit.
Class definition wasn't necessary, because it is already defined on the
debug component itself.
The newer version of the editor correctly handles the plugins. This was
not the case in 0.50.0-beta.5 yet.
@benjay10 benjay10 marked this pull request as draft January 27, 2022 10:21
@benjay10 benjay10 added the enhancement New feature or request label Jan 28, 2022
Changes after a final rebuild of the project, now with the public
beta release of the editor (beta 6).
@benjay10 benjay10 marked this pull request as ready for review January 28, 2022 11:55
Copy link
Collaborator

@lagartoverde lagartoverde left a comment

Choose a reason for hiding this comment

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

good 👍

@benjay10 benjay10 merged commit 63e20e9 into master Feb 2, 2022
@benjay10 benjay10 deleted the feature/use-editor-debug-component-from-GN-3152 branch February 2, 2022 13:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants