Automattic / Automattic/newspack-content-diff-migrator
Comments being later approved don't come over with Content Refresh
- Dominant language
- PHP
- Stars
- 0
- Forks
- 1
- Avg merge
- 5h 6m
- Merged PRs (30d)
- 1
Description
Imagine having post that will be imported with the Content Diff Migrator. If the post has comments that are not still approved, the `comment_count` column for the Post will be 0. [This check](https://github.com/Automattic/newspack-content-diff-migrator/blob/71ee6691c79bd5458a20d648eb5b040e204004c8/src/Logic/ContentDiffMigrator.php#L395) will prevent those comment from being imported.
However, during the double-posting period, those comments can be approved and we will end up with a post without comments.
CC: @iuravic @kariae
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in src/Logic/ContentDiffMigrator.php around the linked comment_count check and trace how Content Refresh decides which comments to import. Confirm the behavior for comments that become approved during the double-posting period; done means the refreshed post includes those comments instead of remaining without them.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100