Skip to content

Conversation

@jrakibi
Copy link
Contributor

@jrakibi jrakibi commented Sep 10, 2025

No description provided.

@adamjonas
Copy link
Collaborator

Status of this?

@jrakibi
Copy link
Contributor Author

jrakibi commented Sep 19, 2025

Status of this?

This PR was just for testing and can be closed. It currently supports nested replies, but only one level of nesting, as I showed in the previous demo.

I’ve updated the way we handle nested replies locally to support multiple levels of nesting on both the scraper and summarizer.

I’ll open two new PRs after completing some local testing with sample data. Once I’m confident they won’t break anything (since I’m not sure we have a backup), we can validate them against our data in Elasticsearch

@jrakibi
Copy link
Contributor Author

jrakibi commented Sep 22, 2025

Closing this PR in favor of #74

The threading functionality is now implemented through these three PRs:

All these three PRs must be merged together.

- We'll perform the data migration on Elasticsearch to update existing documents with the new threading structure
- Merging is not required to test, we can populate/update Elasticsearch by running the scraper job on the branch

@jrakibi jrakibi closed this Sep 22, 2025
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