Skip to content

Fix history not rendering when event time is empty#207

Merged
feedmeapples merged 1 commit intomainfrom
fix-null-timestamp
Jan 11, 2022
Merged

Fix history not rendering when event time is empty#207
feedmeapples merged 1 commit intomainfrom
fix-null-timestamp

Conversation

@feedmeapples
Copy link
Copy Markdown
Contributor

What was changed

Makes null timestamps render w/o throwing an error

Why?

History page was broken when there was an event time == null

Checklist

  1. Closes

  2. How was this tested:

To repro the issue use https://github.com/temporalio/samples-go/tree/main/temporal-fixtures/stuck-workflows
Navigate to the history of workflow with name xxxxxx_stuck_activity

With the fix the page should render

  1. Any docs updates needed?

@feedmeapples feedmeapples merged commit 56c177a into main Jan 11, 2022
@feedmeapples feedmeapples deleted the fix-null-timestamp branch January 11, 2022 17:11
rossnelson added a commit that referenced this pull request Apr 3, 2026
- lodash: ^4.17.21 -> ^4.18.1 (CVE-2026-4800, CVE-2026-2950, CVE-2025-13465)
- svelte: 5.53.3 -> 5.55.1 (CVE-2026-27902, CVE-2026-27901)
- @sveltejs/kit: 2.53.0 -> 2.55.0 (DoS via form deserialization)
- storybook: ^8.6.11 -> ^8.6.18 (CVE-2026-27148)
- tar-fs: >=2.1.2 -> ^3.1.2 (CVE-2025-59343)

Resolves Dependabot alerts #233, #232, #159, #204, #203, #194, #193, #207, #192, #127.
rossnelson added a commit that referenced this pull request Apr 8, 2026
#3269)

- lodash: ^4.17.21 -> ^4.18.1 (CVE-2026-4800, CVE-2026-2950, CVE-2025-13465)
- svelte: 5.53.3 -> 5.55.1 (CVE-2026-27902, CVE-2026-27901)
- @sveltejs/kit: 2.53.0 -> 2.55.0 (DoS via form deserialization)
- storybook: ^8.6.11 -> ^8.6.18 (CVE-2026-27148)
- tar-fs: >=2.1.2 -> ^3.1.2 (CVE-2025-59343)

Resolves Dependabot alerts #233, #232, #159, #204, #203, #194, #193, #207, #192, #127.
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