ninadjeret / wp-notifications-center Goto Github PK

We only have the permalink for default notifications

Pour le moment c'est possible uniquement le titre et non pas pour l'objet

Those custom capabilites will let customize user who can access & use Notifications Center.

https://codex.wordpress.org/Function_Reference/get_option(%27siteurl%27)

When you try to add a variable in the editor you get this error :

Uncaught TypeError: Cannot read property 'execCommand' of null
at HTMLSelectElement. (notification_edit.js?ver=4.9.1:63)

The issue seems to lie with tinymce

jQuery(".voy_mask_select").change(function() {
        
        if( jQuery(":selected", this).val() != "" ) {
            //send_to_editor("{" + jQuery(":selected", this).val() + "}");
            var content_to_send = "{" + jQuery(":selected", this).val() + "}";
            tinymce.activeEditor.execCommand('mceInsertContent', false, content_to_send);
        }       
        jQuery(this).val("");
        return false;
    });

Mettre la bonne variable soit : 'password_reset_link' => wp_lostpassword_url()

In core/template.php, please replace $voynotif_notification->cta by $voynotif_notification->button_info.
Would do a pull request but this repo doesn't seem to be identical to Wordpress plugin directory.

Enable custom HTML template for themes, to override deauflt plugin HTML template

Il manque ce code dans la classe comment_new.php

if( $notification->recipient_type == 'post_author' ) { 
    $author_id = $post->post_author;
    $author_email = get_the_author_meta( 'user_email', $author_id );
    $notification->add_recipient($author_email);
}

Change previsualition settings of many settings from "refresh" to "transport" to improve loading time & user experience

Metter admin_url( 'post.php?post=' . $post->ID ) . '&action=edit' à la place

PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function VOYNOTIF_logs::get_status_title(), 1 passed in /plugins/notifications-center/core/class.logs.php on line 86 and exactly 2 expected in /plugins/notifications-center/core/class.logs.php:436

Dans User_Register ça envoie 'testlink'

Reprendre celui de Wordpress si rien n'est rensiegné dans Notifications Center
Sinon les mails ne s'envoient pas et on a aucune idée pourquoi

Today, Notifications Center stores in DB the default settings used for the email design, such as colors, logos, etc. To do this, all those data are added during plugin activation (& deleted during plugin uninstall).

To avoid useless database queries & storage, it would be better for those default settings to be loaded directly in the plugin files. Only custom data changed by a WP user would be added to the DB.