Skip to content

fix: multiple reloads in timeline#484

Merged
faraazb merged 4 commits intostage_v3from
VE_6673
Aug 6, 2025
Merged

fix: multiple reloads in timeline#484
faraazb merged 4 commits intostage_v3from
VE_6673

Conversation

@contentstackMridul
Copy link
Copy Markdown
Contributor

fix: multiple reloads in timeline
Ticket -> https://contentstack.atlassian.net/browse/VE-6673
Test Report ->
Screenshot 2025-07-11 at 16 43 01

This pull request introduces enhancements to the live preview functionality, focusing on improving query parameter handling, adding support for timeline-specific behavior, and refining event listener logic. The most important changes include updates to query parameter management, adjustments to event handling, and integration of timeline-specific checks.

Query Parameter Handling Enhancements:

  • src/utils/addLivePreviewQueryTags.ts: Added support for the preview_timestamp query parameter and refined logic for adding live_preview, content_type_uid, and entry_uid query parameters.
  • src/utils/index.ts: Improved handling of click events by ensuring query parameters are added only to valid anchor elements and their descendants.

Timeline-Specific Behavior:

Event Listener Refinements:

@contentstackMridul contentstackMridul requested a review from a team as a code owner August 6, 2025 06:39
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 6, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 71.41% 8743 / 12243
🔵 Statements 71.41% 8743 / 12243
🔵 Functions 71.46% 313 / 438
🔵 Branches 84.07% 1109 / 1319
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/livePreview/eventManager/postMessageEvent.hooks.ts 83.69% 76.47% 75% 83.69% 34-36, 83-84, 97, 106, 119-127
src/utils/addLivePreviewQueryTags.ts 74.07% 50% 100% 74.07% 13-14, 16-18, 20-21
src/utils/index.ts 89.74% 85.71% 100% 89.74% 36-37, 45-46
Generated in workflow #502 for commit e781891 by the Vitest Coverage Report Action

@faraazb faraazb merged commit 7695da3 into stage_v3 Aug 6, 2025
15 of 17 checks passed
@faraazb faraazb deleted the VE_6673 branch August 6, 2025 08:39
@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.

2 participants