Skip to content

fix: added update to stop the double firing of init in vb#600

Merged
contentstackMridul merged 4 commits into
develop_v4from
VB-1667
May 29, 2026
Merged

fix: added update to stop the double firing of init in vb#600
contentstackMridul merged 4 commits into
develop_v4from
VB-1667

Conversation

@contentstackMridul
Copy link
Copy Markdown
Contributor

fix: added update to stop the double firing of init in vb

@contentstackMridul contentstackMridul requested review from a team as code owners May 26, 2026 06:44
@github-actions
Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 26, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 66.58% 2483 / 3729
🔵 Statements 65.45% 2522 / 3853
🔵 Functions 64.17% 446 / 695
🔵 Branches 60.76% 1490 / 2452
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/common/inIframe.ts 90.9% 75% 100% 90.9% 15
src/configManager/configManager.ts 100% 100% 100% 100%
src/livePreview/eventManager/postMessageEvent.hooks.ts 98.57% 96.72% 90% 98.57% 157
Generated in workflow #834 for commit d3d2a09 by the Vitest Coverage Report Action

Copy link
Copy Markdown
Contributor

@hitesh-shetty-cstk hitesh-shetty-cstk left a comment

Choose a reason for hiding this comment

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

LGTM!!

fix: fixed the sideEffect issue due to setConfigFromParams
@github-actions
Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

@contentstackMridul contentstackMridul merged commit a46d025 into develop_v4 May 29, 2026
8 checks passed
@contentstackMridul contentstackMridul deleted the VB-1667 branch May 29, 2026 11:13
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.

2 participants