Skip to content

Conversation

aristath
Copy link
Member

@aristath aristath commented Sep 22, 2025

Detects site-url changes.
If the site URL has changed, makes a request to a new endpoint on our server.
That endpoint then updates the entry in our database for the newsletters etc.
Of course that only happens if the site is already registered to get newsletters.

Depends on https://github.com/ProgressPlanner/progress-planner-server/pull/47

@aristath aristath marked this pull request as draft September 22, 2025 08:44
Copy link
Contributor

github-actions bot commented Sep 22, 2025

Test on Playground
Test this pull request on the Playground
or download the zip

@aristath aristath marked this pull request as ready for review September 22, 2025 11:16
Base automatically changed from ari/auto-register-if-branded to develop September 24, 2025 05:10
@aristath aristath requested a review from Copilot October 16, 2025 09:49
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements site URL change detection functionality to notify the remote server when a WordPress site's URL has changed. This enables automatic updates to the database for newsletter registrations and other site-dependent data.

Key changes:

  • Added site URL change detection that runs on WordPress shutdown hook
  • Refactored remote URL handling to use a single parameterized method
  • Updated references to use the new consolidated URL method

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
classes/utils/class-onboard.php Added site URL change detection method and consolidated remote URL handling
classes/admin/class-page.php Updated to use the new parameterized remote URL method

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@aristath aristath merged commit 1466073 into develop Oct 21, 2025
28 checks passed
@aristath aristath deleted the ari/detect-domain-changes branch October 21, 2025 11:15
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