#161 - Allow defining support monitor constants to override settings.

[joshuaabenazer]

Description of the Change

Allow defining support monitor constants to override settings. Support has been added for both network activation and single-site activation.
New constants that were introduced.

1. SUPPORT_MONITOR_ENABLE
2. SUPPORT_MONITOR_API_KEY
3. SUPPORT_MONITOR_SERVER_URL

Closes #161

Default View:

View with Constants Set

define( 'SUPPORT_MONITOR_ENABLE', 'yes' );
define( 'SUPPORT_MONITOR_API_KEY', '123123123145' );
define( 'SUPPORT_MONITOR_SERVER_URL', 'https://example.org' );

How to test the Change

1. Test out the settings for support monitor settings as part of this plugin on both a network activated multisite installation and a single site installation.
2. Check by defining various combinations of all or some of the constants in the wp-config.php and reviewing the Support Monitor Settings. They should replace the user saved values and also not be editable when it is defined.
3. Removing all or any of the constants from the wp-config.php should preserve the prior user saved values for the settings.

Changelog Entry

Added - New Support Monitor settings constants - SUPPORT_MONITOR_ENABLE, SUPPORT_MONITOR_API_KEY, SUPPORT_MONITOR_SERVER_URL

Credits

Props @joshuaabenazer, @darylldoyle, @claytoncollie

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[darylldoyle]

@claytoncollie not sure if you want to test this one out?

[claytoncollie]

@darylldoyle I pulled this branch into my project and it worked perfectly. Fields are being overwritten and set to disabled.

Thank you @joshuaabenazer for the effort here.

For reference, here are the definitions I used in my wp-config.php file.

define( 'SUPPORT_MONITOR_ENABLE', 'yes' );
define( 'SUPPORT_MONITOR_API_KEY', 'api key goes here' );
define( 'SUPPORT_MONITOR_SERVER_URL', 'https://google.com' );

[darylldoyle]

@joshuaabenazer nice work here!

Can we please update the UI when the values are set via constants? Currently when I set SUPPORT_MONITOR_ENABLE to yes in the cost, it still shows as No in the UI.

The same goes for the API Key and URL, the API key can be masked, but it'd be nice to show that there's a value set.

I've made updates to this PR, so dismissing my own review from before