bccampus / custom-wp-notify Goto Github PK

Description

One of the dependencies is throwing PHP Parsing Errors, preventing cron from running and notifications from being sent out.

Expected behaviour

Vendor dependencies should work in different server environments

Actual behaviour

vendor/symfony/css-selectors required by tijsverkoyen/css-to-inline-styles is causing problems on cert (PHP 7.0). No such problems occur on local development (PHP 7.1)

function https://github.com/BCcampus/custom-wp-notify/blob/dev/inc/processors/class-mail.php#L128 in Cron hook produces the following on cert server:

Steps to reproduce the problem

Force the Mail Queue to run

System Information

• Custom WP Notify: pre-release
• WordPress: 4.9.4

Description

Javascript in a php file class-cwpoptions.php prevents code formatting (for PHP) from working. JS needs to be moved out into its own file(s).

Expected behaviour

PHP files should respond to code formatting actions

Actual behaviour

They don't

Description

Upon activation the Subscription Management admin interface is empty.

Expected behaviour

On plugin activation, all users, if not already set, should have their cwp_notify value in user_meta set to 0, or off.

Actual behaviour

This value is not set

Steps to reproduce the problem

Activate the plugin, go to Tools->Custom WP Notify and click on the 'Subscription Management' tab

System Information

• Custom WP Notify: 0.5.0
• WordPress: 4.9.5

Description

shifting the responsibility of dependency mgmt to dependency managers.

Expected behaviour

automatic updates to dependencies could be better managed with yarn update

Actual behaviour

requires manually checking on packages and then updating version number in a CDN link.

Steps to reproduce the problem

• bootstrap https://github.com/BCcampus/custom-wp-notify/blob/dev/inc/class-cwpoptions.php#L54
• multiselect https://github.com/BCcampus/custom-wp-notify/blob/dev/inc/class-cwpoptions.php#L52

System Information

• Custom WP Notify: 0.6.0
• WordPress: 5.0.3

Can't move users between the "Subscribed" and "Not Subscribed" select lists.

[Description of the bug or feature]

Moving a user back and forth between subscribed and non subscribed should work.

[What you expect/want to happen]

Nothing happens when you try and move a user.

[What actually happened]

Users stay in the same list.

1. Login to eypd.bccampus.ca with an account with admin privileges
2. Visit the Custom WP notify settings page (Settings -> Custom WP Notify)
3. Click on the "Subscription Management" tab
4. Try to move a user over

System Information

• Custom WP Notify: v0.6.0
• WordPress: 4.9.7