Skip to content

fix: removed edit icon when the click is on the container#478

Merged
SahilCs15 merged 1 commit intodevelop_v3from
VE-7130
Aug 4, 2025
Merged

fix: removed edit icon when the click is on the container#478
SahilCs15 merged 1 commit intodevelop_v3from
VE-7130

Conversation

@SahilCs15
Copy link
Copy Markdown
Contributor

This PR fixes the issue where a 'Something went wrong' error appeared when opening the field edit modal, specifically when the JSON RTE container was selected.

@SahilCs15 SahilCs15 requested a review from a team as a code owner August 4, 2025 13:57
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 4, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 72.15% 9091 / 12599
🔵 Statements 72.15% 9091 / 12599
🔵 Functions 71.86% 327 / 455
🔵 Branches 84.24% 1134 / 1346
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/visualBuilder/components/FieldToolbar.tsx 77.16% 77.08% 37.5% 77.16% 64-112, 134, 173, 208-225, 246-249, 272-281, 310-318, 326-327, 330-331, 383-389, 408-409, 473, 510, 591-598
Generated in workflow #495 for commit 4c06df4 by the Vitest Coverage Report Action

@SahilCs15 SahilCs15 requested a review from faraazb August 4, 2025 15:07
Copy link
Copy Markdown
Contributor

@hiteshshetty-dev hiteshshetty-dev left a comment

Choose a reason for hiding this comment

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

LGTM!!

@SahilCs15 SahilCs15 merged commit 13ace62 into develop_v3 Aug 4, 2025
9 of 11 checks passed
@SahilCs15 SahilCs15 deleted the VE-7130 branch August 4, 2025 16:41
@SahilCs15 SahilCs15 restored the VE-7130 branch August 6, 2025 06:35
faraazb added a commit that referenced this pull request Aug 6, 2025
fix: removed edit icon when the click is on the container (#478)
@faraazb faraazb mentioned this pull request Aug 6, 2025
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.

3 participants