Skip to content

Feat/modify sync button #3536

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Feat/modify sync button #3536

wants to merge 30 commits into from

Conversation

rithikb24
Copy link
Contributor

@rithikb24 rithikb24 commented Jul 30, 2025 •
edited
Loading

Description

A "Sync Products" button has been modified to sync modified products first and then sync the entire product catalog. This feature will accelerate synchronisation by only syncing product changes made recently first.

This will particularly help merchants with large product catalogs

  • Track Last Sync Time: Utilize _fb_sync_last_time within WordPress post meta to record the last synchronization time for each product. This data will be stored in the wp_postmeta table.
  • Enable Delta Sync: Call a function to sync delta of products then sync the catalog.
  • Compare Timestamps: Product modification timestamps will be compared against their last sync timestamps.
  • Queue New/Modified Products: Only products that are new or have been modified since their last synchronization will be queued for the next sync.

Checklist

  • ✅ I have commented my code, particularly in hard-to-understand areas, if any.
  • ✅ I have confirmed that my changes do not introduce any new PHPCS warnings or errors.
  • ✅ I have checked plugin debug logs that my changes do not introduce any new PHP warnings or FATAL errors.
  • ✅ I followed general Pull Request best practices. Meta employees to follow this wiki.
  • [] I have added tests (if necessary) and all the new and existing unit tests pass locally with my changes.
  • [In Progress] I have completed dogfooding and QA testing, or I have conducted thorough due diligence to ensure that it does not break existing functionality.
  • [] I have updated or requested update to plugin documentations (if necessary). Meta employees to follow this wiki.

Changelog entry

One liner entry to be surfaced in changelog.txt

Test Plan

Steps followed to test changes [Dogfooding Doc]

@meta-cla meta-cla bot added the CLA Signed label Jul 30, 2025
@rithikb24 rithikb24 marked this pull request as ready for review August 1, 2025 12:35
@rithikb24 rithikb24 added the changelog: add A new feature, function, or functionality was added. label Aug 1, 2025
@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

rithikb24 and others added 3 commits August 6, 2025 15:33
@rithikb24
feat: add help tips
d19f13b
@rithikb24
Feat/sync changes (#1)
089c328 
* feat: sync from the same button

* chore: fix files

* chore: remove older test file
@rithikb24
chore: remove help tip
683860c
@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

@facebook-github-bot
Copy link
Contributor

@rithikb24 has imported this pull request. If you are a Meta employee, you can view this in D79439754.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
changelog: add A new feature, function, or functionality was added. CLA Signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rithikb24 @facebook-github-bot