Skip to content

Report preview timestamp parse errors#1085

Closed
dreikanter wants to merge 1 commit into
mainfrom
agent/f066-report-preview-parse-errors
Closed

Report preview timestamp parse errors#1085
dreikanter wants to merge 1 commit into
mainfrom
agent/f066-report-preview-parse-errors

Conversation

@dreikanter

Copy link
Copy Markdown
Owner

Changes:

  • Report malformed preview timestamps through Rails.error instead of writing an isolated warning log.
  • Keep returning nil so an invalid timestamp does not break preview rendering.

Why:

Handled exceptions should still reach the application's error-reporting pipeline.

References:

Checks:

  • User-facing fallback behavior is unchanged.
  • GitHub Actions will verify the branch.

@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 97.55%. Comparing base (ce0547a) to head (1134896).

Files with missing lines Patch % Lines
app/components/post_preview_component.rb 0.00% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1085   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         355      355           
  Lines        8358     8358           
=======================================
  Hits         8154     8154           
  Misses        204      204           
Files with missing lines Coverage Δ
app/components/post_preview_component.rb 92.20% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant