Skip to content

fix: History page filter error when initial event is selected#1100

Merged
Assem-Uber merged 1 commit intocadence-workflow:masterfrom
Assem-Uber:fix/17013/filter-error-when-initial-event-is-selected
Dec 2, 2025
Merged

fix: History page filter error when initial event is selected#1100
Assem-Uber merged 1 commit intocadence-workflow:masterfrom
Assem-Uber:fix/17013/filter-error-when-initial-event-is-selected

Conversation

@Assem-Uber
Copy link
Contributor

@Assem-Uber Assem-Uber commented Dec 2, 2025

Summary
History page errors out when an event exists in the url and then filters is change to a value that filters out this event.

Screenshot 2025-12-02 at 14 12 53

Fix
Tolerat the absense of selected event in filteredEventGroupsEntries by checking if it exists before accessing sub fields.

Tests

  • Added test case to cover the scenario
  • Verified that changing filters to execlude the initial selected event doesn't error out.

Signed-off-by: Assem Hafez <assem.hafez@uber.com>
@Assem-Uber Assem-Uber merged commit 0f2f788 into cadence-workflow:master Dec 2, 2025
4 checks passed
@Assem-Uber Assem-Uber deleted the fix/17013/filter-error-when-initial-event-is-selected branch December 2, 2025 16:30
adhityamamallan pushed a commit to adhityamamallan/cadence-web that referenced this pull request Dec 3, 2025
Signed-off-by: Assem Hafez <assem.hafez@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
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