Skip to content

fix(timestamps): keep links working after played#169

Merged
chhoumann merged 1 commit into
masterfrom
fix/166-played-timestamp-links
May 2, 2026
Merged

fix(timestamps): keep links working after played#169
chhoumann merged 1 commit into
masterfrom
fix/166-played-timestamp-links

Conversation

@chhoumann
Copy link
Copy Markdown
Owner

Keep timestamp URI clicks from being overridden by a saved finished playback position.

When a timestamp link loads an episode that has already been marked as played, PodNotes now carries the requested timestamp through to the player and applies it after media metadata loads, before restoring saved playback progress. Already-visible linked episodes also seek and resume immediately. This also preserves time=0 timestamp links when generating PodNotes URIs.

Focused tests cover the URI handler handoff and the player restore path for finished episodes. I also verified the fix in the Obsidian dev vault with a linked timestamp after marking the episode played.

Fixes #166

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 2, 2026

Deploying podnotes with  Cloudflare Pages  Cloudflare Pages

Latest commit: ac05f57
Status: ✅  Deploy successful!
Preview URL: https://81282066.podnotes.pages.dev
Branch Preview URL: https://fix-166-played-timestamp-lin.podnotes.pages.dev

View logs

@chhoumann chhoumann marked this pull request as ready for review May 2, 2026 15:48
chatgpt-codex-connector[bot]

This comment was marked as resolved.

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

@chhoumann chhoumann force-pushed the fix/166-played-timestamp-links branch from e042ed7 to ac05f57 Compare May 2, 2026 16:03
@chhoumann chhoumann merged commit d094eae into master May 2, 2026
2 checks passed
@chhoumann chhoumann deleted the fix/166-played-timestamp-links branch May 2, 2026 16:08
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

🎉 This PR is included in version 2.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Timestamp links in notes stop working after episode is marked as played

1 participant