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

EB-1201: Edit button is showing in live editor window #112

Conversation

Vishvam10
Copy link
Contributor

Jira Story

Work has been completed on this Jira story : https://contentstack.atlassian.net/browse/EB-1201

Overview

This PR fixes a bug where the edit button appeared in the live editor. It introduces and takes the help of Preact Effects for better state updation

Changes Made

  • Add effect() and update rendering logic in editButton.ts
  • Update relevant test cases

@Vishvam10 Vishvam10 added the bug Something isn't working label Mar 12, 2024
@Vishvam10 Vishvam10 requested a review from Deepak-Kharah March 12, 2024 08:02
@Vishvam10 Vishvam10 self-assigned this Mar 12, 2024
@Vishvam10 Vishvam10 requested a review from a team as a code owner March 12, 2024 08:02
@Vishvam10 Vishvam10 closed this Mar 12, 2024
@Vishvam10 Vishvam10 deleted the EB-1201-edit-button-is-showing-in-live-editor-window branch March 12, 2024 08:03
@Vishvam10 Vishvam10 restored the EB-1201-edit-button-is-showing-in-live-editor-window branch March 12, 2024 08:03
@Vishvam10 Vishvam10 reopened this Mar 12, 2024
Copy link
Contributor

@Deepak-Kharah Deepak-Kharah left a comment

Choose a reason for hiding this comment

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

Let's add the missing unit test cases for these. Let's update the E2E ticket as we need to check some of the flows there.

@Vishvam10 Vishvam10 requested a review from Deepak-Kharah March 12, 2024 13:11
@Vishvam10 Vishvam10 merged commit 98f4452 into VC-115/live-editor-support Mar 12, 2024
3 checks passed
@aravindbuilt aravindbuilt deleted the EB-1201-edit-button-is-showing-in-live-editor-window branch January 15, 2025 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants