GitBulletin

Repository: farcasterxyz/hub-monorepo

Revolution in Event Management: New Timestamp Function Set to Transform Snapchain Accuracy!

2025-06-11 22:42:40 UTC

Developers unveil groundbreaking fix, deprecating old functions in favor of enhanced precision and performance.

Commit Details:

fix: Fix event timestamp calculation for snapchain (#2598) ## Why is this change needed? Deprecate existing event timestamp functions that rely on event id, and add a new one that fetch it from the itself itself. ## Merge Checklist _Choose all relevant options below by adding an `x` now or at any time before submitting for review_ - [x] PR title adheres to the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) standard - [x] PR has a [changeset](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#35-adding-changesets) - [x] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore) - [ ] PR includes [documentation](https://github.com/farcasterxyz/hub-monorepo/blob/main/CONTRIBUTING.md#32-writing-docs) if necessary. <!-- start pr-codex --> --- ## PR-Codex overview This PR focuses on updating the event timestamp handling in the `shuttle` and `core` packages. It replaces the deprecated `extractEventTimestamp` function with `extractTimestampFromEvent`, which aligns with the new event structure using timestamps instead of event IDs. ### Detailed summary - Changed function name from `extractEventTimestamp` to `extractTimestampFromEvent` in `hubSubscriber.ts` and `eventStream.ts`. - Updated calls to `extractEventTimestamp` to use `extractTimestampFromEvent`. - Marked `extractEventTimestamp` as deprecated in `time.ts` with a note on using event timestamps. - Added a new implementation for `extractTimestampFromEvent` in `time.ts`. > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->

Search Similar