Skip to content

fix: include all events in timeline regardless of parent status#23

Open
mangesh3107 wants to merge 1 commit intomaciejwalkowiak:mainfrom
mangesh3107:feature/missing-parent-event
Open

fix: include all events in timeline regardless of parent status#23
mangesh3107 wants to merge 1 commit intomaciejwalkowiak:mainfrom
mangesh3107:feature/missing-parent-event

Conversation

@mangesh3107
Copy link
Copy Markdown

Encountered a use-case :

  • parent events were missing in timeline but being referred in other events, this led to incomplete report generation.

Changes:

  • Remove filter that excluded events with parent IDs, ensuring orphaned events are preserved when parent events are missing from timeline.
  • Updated existing test to support new behaviour
  • Added new test for missing parent event use-case

Remove filter that excluded events with parent IDs, ensuring orphaned
events are preserved when parent events are missing from timeline.
@maciejwalkowiak
Copy link
Copy Markdown
Owner

Thanks for PR @mangesh3107!

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