Skip to content

Comments

Prefer timestamp from ddb record#406

Merged
petermyers merged 3 commits intomasterfrom
issue-ddb-timestamp
May 18, 2025
Merged

Prefer timestamp from ddb record#406
petermyers merged 3 commits intomasterfrom
issue-ddb-timestamp

Conversation

@petermyers
Copy link
Collaborator

@petermyers petermyers commented May 14, 2025

  • Updates fromDynamodb to prefer to timestamp from the stored record for uow.event.timestamp rather than the approximate creation time.
  • If no NewImage on record, still prefer approximate creation time.
  • Bumps version to 1.1.0 simply because this is sort of a big semantic change for trigger functions. Open to discussion / just incrementing bug version if we'd prefer.

@petermyers petermyers requested a review from jgilbert01 May 14, 2025 18:50
@petermyers petermyers requested a review from jgilbert01 May 15, 2025 17:25
@petermyers petermyers merged commit a4fbc8f into master May 18, 2025
2 checks passed
@petermyers petermyers deleted the issue-ddb-timestamp branch May 18, 2025 22:57
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