(DP41) Mod Action Notify

[abobader]

Hi @DawPi 

Still with every update/patch I see the this:

Thanks.

[DawPi]

4 hours ago, abobader said:

Hi @DawPi 

Still with every update/patch I see the this:

Thanks.

What is that? Is that old plugin, I think?

New installation (app):

 

[abobader]

12 hours ago, DawPi said:

What is that? Is that old plugin, I think?

It the latest app build:

[DawPi]

Send me access details, I'll take a look.

[abobader]

14 hours ago, DawPi said:

Send me access details, I'll take a look.

PM sent, thanks in advance.

[DawPi]

On 12/15/2022 at 11:47 PM, abobader said:

It the latest app build:

Well, this is a screen from your board:

 

So everything seems fine for me as you see. Maybe it's some kind of "error/bug" on IPS4 side? Not sure.

[abobader]

28 minutes ago, DawPi said:

So everything seems fine for me as you see. Maybe it's some kind of "error/bug" on IPS4 side? Not sure.

Seem like it, thanks for looking to this matters.

[Dll]

If you have this set to send a pm from a different user to the one who took the action, it messages when someone hides or edits their own posts. Any chance this could be changed in an upcoming version?

[DawPi]

Hm, I'll take a look soon and let you know.

Hm, it should skip it:

            /* Do not send if author is doing mod action */
            if( $item->mapped('author') == \IPS\Member::loggedIn()->member_id )

 

Could you specify step by step how to reproduce it?

[Dll]

Not really many steps to re-produce. Set the author of the message which is sent to a third party, do something with your own post which should trigger a message, and it'll send you one. 

[DawPi]

Can't confirm. What version are you using?

[Dll]

The latest one. Do you have it set to just send a message, not online or email?

[Gabriel Torres]

@DawPi

I've just installed your app and I have a few suggestions:

1. Replacement strings (tags) for the title. As it happens, right now all private messages have the same title, which is not the ideal. The user who has three different topics/posts moderated, will receive three messages with the same title and might think they are simply duplicates. Also, in the future, it might be difficult to locate a specific message.

2. For the move action, to add the source and target forums.

3. The content editor could use the Rich Text editor

Thanks!

Edited June 29, 2023 by Gabriel Torres

[DawPi]

2 hours ago, Gabriel Torres said:

1. Replacement strings (tags) for the title. As it happens, right now all private messages have the same title, which is not the ideal. The user who has three different topics/posts moderated, will receive three messages with the same title and might think they are simply duplicates. Also, in the future, it might be difficult to locate a specific message.

Hm, any suggestions about these tags?

2 hours ago, Gabriel Torres said:

2. For the move action, to add the source and target forums.

Below the PW content? 🙂

 

2 hours ago, Gabriel Torres said:

3. The content editor could use the Rich Text editor

I'll take a look into this.

 

Thanks!

[Gabriel Torres]

29 minutes ago, DawPi said:

Hm, any suggestions about these tags?

Ideally something along the lines of:

Your topic has been moved: <title>

Your topic has been edited: <title>

Your post has been hidden: <title>

Where <title> is the topic's title.

31 minutes ago, DawPi said:

Below the PW content? 🙂

It could be incorporated in the #reason_stuff# tag.

Currently: John Doe moved a topic <title>

Proposal: John Doe moved a topic <title> from <origin> to <destination>

Where <origin> and <destination> are the forums names with links to them.

Thanks for the reply! 🙂

[DawPi]

We will see what I can do with that. 😉

[DawPi]

On 6/29/2023 at 8:22 PM, Gabriel Torres said:

1. Replacement strings (tags) for the title. As it happens, right now all private messages have the same title, which is not the ideal. The user who has three different topics/posts moderated, will receive three messages with the same title and might think they are simply duplicates. Also, in the future, it might be difficult to locate a specific message.

Better? 🙂

 

 

 

Taken few actiona and the result:

 

 

 

New title:

Result:

 

 

Hope it's fine? 🙂

 

On 6/29/2023 at 8:22 PM, Gabriel Torres said:

2. For the move action, to add the source and target forums.

Unfortunatelly it isn't doable. My mod hook into modLog which is perform AFTER move (source and destination forums would be the same):

 

On 6/29/2023 at 8:22 PM, Gabriel Torres said:

3. The content editor could use the Rich Text editor

Which editor?

 

[Gabriel Torres]

5 hours ago, DawPi said:

Better? 🙂

It helps, thanks.

5 hours ago, DawPi said:

Which editor?

The standard WYSIWYG editor from the platform instead of a textarea field.

Cheers.

[DawPi]

1 hour ago, Gabriel Torres said:

The standard WYSIWYG editor from the platform instead of a textarea field.

Where? 🙂

[Gabriel Torres]

@DawPi

You current approach: simple textarea

Best approach: WYSIWYG editor

[DawPi]

Done. Waiting on approval. 🙂

 

[Gabriel Torres]

@DawPi Installed the latest version and when adding the content, I missed a tag for the user's name! So we can write something more personalized like:

Quote

Hi, #username#!

(...)

Another issue I faced is that when a post is hidden, no notification is generated. The notification is generated only when a whole topic is hidden. You may want to take a look into this.

Thanks.

[DawPi]

35 minutes ago, Gabriel Torres said:

and when adding the content, I missed a tag for the user's name! So we can write something more personalized like

It wasn't there in previous version too so what you missed exactly? 🙂

 

36 minutes ago, Gabriel Torres said:

Another issue I faced is that when a post is hidden, no notification is generated. The notification is generated only when a whole topic is hidden. You may want to take a look into this.

Can't confirm:

 

 

[Gabriel Torres]

Just now, DawPi said:

It wasn't there in previous version too so what you missed exactly? 🙂

I know that it wasn't present before. I am suggesting you to add this is a future version.

1 minute ago, DawPi said:

Can't confirm:

Thanks, I will test this again! I may have missed something here.

[abobader]

Hi @DawPi 

One notice, when normally want to check for app/plugin update from:

System - Application - Check for updates

This app do not show any indicated for update, so when I saw you update the app and still not notify on the admincp as I said, only way was to click on the app and then click on "Marketplace" then you will see the update and can update this app.

I notice this some time ago but now I can confirm the issue.

Thanks.