Fix / 9130 Changes for Approval Into Main

[10upsimon]

Summary

Addresses issue:

• Create Scheduled_Events_Check Class #9130

Relevant technical choices

• Updated code and renamed properties to be indicative of events_sync as opposed to events_check
• Updated inline comments, var declarations and property references to match changes above

PR Author Checklist

• My code is tested and passes existing unit tests.
• My code has an appropriate set of unit tests which all pass.
• My code is backward-compatible with WordPress 5.2 and PHP 7.4.
• My code follows the WordPress coding standards.
• My code has proper inline documentation.
• I have added a QA Brief on the issue linked above.
• I have signed the Contributor License Agreement (see https://cla.developers.google.com/).

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Code Reviewer Checklist

• Run the code.
• Ensure the acceptance criteria are satisfied.
• Reassess the implementation with the IB.
• Ensure no unrelated changes are included.
• Ensure CI checks pass.
• Check Storybook where applicable.
• Ensure there is a QA Brief.

Merge Reviewer Checklist

• Ensure the PR has the correct target branch.
• Double-check that the PR is okay to be merged.
• Ensure the corresponding issue has a ZenHub release assigned.
• Add a changelog message to the issue.

[github-actions]

Build files for 1c0199f have been deleted.

[aaemnnosttv]

Thanks @10upsimon. One more thing, can we rename Conversion_Reporting_Events_Sync::check_for_events to sync_detected_events? I think that would be more clear as to what it's doing and more consistent with the naming.