Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DS-329 Remove Button from Micro Journey Editor #2294

Merged
merged 6 commits into from
Aug 11, 2021

Conversation

cjwhitedev
Copy link
Collaborator

Jira

https://pegadigitalit.atlassian.net/browse/DS-329

Summary

Removed Bolt-Button option from Micro Journey Editor

Details

Bolt-Button has been deprecated and slated for removal. Micro Journeys is also to be removed eventually. So rather than resolve any issues one might cause the other, we thought it best just to remove the button from the editor.

How to test

Review my code

Release notes

Removed Bolt-Button option from Micro Journey Editor

@github-actions github-actions bot added the type: feature List this PR in the 'Features' section of the release notes. label Aug 6, 2021
Copy link
Collaborator

@danielamorse danielamorse left a comment

Choose a reason for hiding this comment

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

AFAICT there is no actual usage of bolt-button in micro journeys content, so this should be ok. Having Ross verify that is also reassuring.

@remydenton does WWO use this same editor? If so, should @cjwhitedev look at WWO before we make this change?

Copy link
Collaborator

@remydenton remydenton left a comment

Choose a reason for hiding this comment

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

This looks great, thanks for the updates @cjwhitedev. I'm glad we realized we could just remove rather than sinking more time into making it work.

@danielamorse , WWO actually doesn't use the editor. WWO instances are created with fixed markup written by a developer, not a content author. It means a little less flexibility, but also way less complexity.

@remydenton remydenton merged commit 958fc5a into master Aug 11, 2021
@remydenton remydenton deleted the feature/DS-329-grapejs-button-color branch August 11, 2021 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature List this PR in the 'Features' section of the release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants