Skip to content

VB-476:- fix(VB-476/open-experience-editig-message-clear): message updated#501

Merged
csAdityaPachauri merged 5 commits into1st-oct-2025from
VB-476/open-experience-editig-message-clear-developv4-sync
Sep 22, 2025
Merged

VB-476:- fix(VB-476/open-experience-editig-message-clear): message updated#501
csAdityaPachauri merged 5 commits into1st-oct-2025from
VB-476/open-experience-editig-message-clear-developv4-sync

Conversation

@csAdityaPachauri
Copy link
Copy Markdown
Contributor

@csAdityaPachauri csAdityaPachauri commented Sep 15, 2025

MeWhen we open an experience and try to edit anything this message is shown - “Open An Experience from Audience Widget to start editing” which is not very clear. User has already opened audience widget and selected audiences to reach that variant. The edit button on the Experience in side panel is for editing the content in canvas is not very clearly mentioned anywhere.
Screenshot 2025-09-15 at 1 47 23 PM

@csAdityaPachauri csAdityaPachauri requested a review from a team as a code owner September 15, 2025 05:58
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 15, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 72.43% 9288 / 12823
🔵 Statements 72.43% 9288 / 12823
🔵 Functions 72.23% 333 / 461
🔵 Branches 84.14% 1173 / 1394
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/visualBuilder/utils/isFieldDisabled.ts 100% 97.61% 100% 100%
Generated in workflow #557 for commit 21c85d1 by the Vitest Coverage Report Action

Base automatically changed from develop_v4 to stage_v4 September 16, 2025 17:58
@KANE-99 KANE-99 requested a review from a team as a code owner September 16, 2025 17:58
csAyushDubey
csAyushDubey previously approved these changes Sep 18, 2025
Copy link
Copy Markdown
Contributor

@csAyushDubey csAyushDubey left a comment

Choose a reason for hiding this comment

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

LGTM

faraazb
faraazb previously approved these changes Sep 18, 2025
Copy link
Copy Markdown
Contributor

@faraazb faraazb left a comment

Choose a reason for hiding this comment

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

LGTM!

Base automatically changed from stage_v4 to main September 18, 2025 12:12
@csAdityaPachauri csAdityaPachauri dismissed stale reviews from faraazb and csAyushDubey September 18, 2025 12:12

The base branch was changed.

@csAdityaPachauri csAdityaPachauri changed the base branch from main to 1st-oct-2025 September 21, 2025 17:02
Copy link
Copy Markdown
Contributor

@karancs06 karancs06 left a comment

Choose a reason for hiding this comment

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

Lgtm

@csAdityaPachauri csAdityaPachauri merged commit 0df93a5 into 1st-oct-2025 Sep 22, 2025
10 checks passed
@csAdityaPachauri csAdityaPachauri deleted the VB-476/open-experience-editig-message-clear-developv4-sync branch September 22, 2025 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants