Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CDiff multiple source sites to same destination #493

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

iuravic
Copy link
Collaborator

@iuravic iuravic commented Apr 28, 2024

A work in progress PR to add CDiff support for multibranded sites, or other sites which need merging of two different source sites to one destination.

Description

Two parts to this PR:

  • update postmeta with original ID to contain site hostname -- this is now complete at the time of submitting this WIP PR
  • update the content-diff-migrate-live-content command to use postmeta instead of log files with migrated IDs, and distinguish between multiple sources -- not done yet

Testing

Detailed testing is necessary, and I would propose doing this scenario:

  • set up 3 sites, two sources and one destination
  • create posts on source sites:
    • some posts which have different/unique IDs among themselves
    • some posts which use identical IDs (e.g. post ID 100 on first site, and a different post on second site but also has ID 100)
    • have those posts use featured images, again some thumbnail images with same post (attachment) IDs and others with unique
    • run a first content diff import
      • check all the restults -- posts integrity, featured images, media library
    • continue towards a second content refresh by setting up the two source sites more
      • add more posts -- also some with new same and new unique IDs, and thumbnails)
      • modify some existing posts -- change the featured image on some
      • import posts and check all newly added and modified data, including existing post added in first round

  • confirmed that PHPCS has been run

@iuravic iuravic self-assigned this Apr 28, 2024
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