Skip to content

Conversation

@jpetto
Copy link
Collaborator

@jpetto jpetto commented Dec 10, 2025

Goal

remove parser dependency

  • removes time to read (not available from zyte)
  • removes publish date (ticket filed to re-add)

Todos

  • test locally against dev data
  • test on dev

Reference

Tickets:

Implementation Decisions

commented out code that we'll need again once publish date is added to the prospect object.

@jpetto jpetto requested a review from a team as a code owner December 10, 2025 20:48
Copy link
Contributor

@mmiermans mmiermans left a comment

Choose a reason for hiding this comment

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

lgtm

- removes time to read (not available from zyte)
- removes publish date (ticket filed to re-add)
- retrieves datePublished from the graph
- re-enables display of and filtering by datePublished
@jpetto jpetto force-pushed the HNT-1363-remove-parser-dependency branch from 2eb4714 to bd0166a Compare December 11, 2025 18:51
@jpetto jpetto merged commit 6589a1a into main Dec 11, 2025
6 checks passed
@jpetto jpetto deleted the HNT-1363-remove-parser-dependency branch December 11, 2025 20:35
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.

3 participants