Mark Notification Download 6.1.2.0

5.0
us Michieal ~ Coder ~ 8 months ago

This component adds in the ability to mark a notification as read, or to delete a notification without leaving the page that you are on. No more having to click the various notifications to mark them as read. This utility grants the user the ability to delete their own notifications (and thus help keep your database clean). Modeled off of how Facebook allows you to just mark a notification as read, or to delete those annoying "this person liked your comment" notifications.

Suggestions for improvement, or bug fixes (or simply finding one and letting me know) are always welcome!

This is the initial release of the component, and I have tested it on a live site.

Comments
us Pariter Retirap Replied 5 months ago

This is an example of notifications that don't get deleted "out of the box" with this component. Have not attempted to bug fix this yet, I imagine it's just because Videos/group/event haven't been tagged as an item to receive the action button. Thanks for your hard work!

enter image description here

German Michael Zülsdorff Replied 6 months ago

Just tried 6.1.2.0, and the logic appears still strange to me. This screenshot shows two groupies of mine:

  • Honey, who liked a photo some days ago. This notification has been marked already as read and because of that it is no longer part of popup on the right
  • Botox, who just added a fresh comment. And this noticfiation is in fact still unread.

Nevertheless, I get the same 'Mark Read' link in both cases - I would expect to get it with Botox, only.


enter image description here

us Michieal ~ Coder ~ Replied 7 months ago

As seen in the image below, only the first notification goes through the templating process, as defined in view.php. (I copied over the original into my component so that I could modify it.)

Notification Template Issue

us Michieal ~ Coder ~ Replied 7 months ago

@Hugo -- Thank you! I added it in.

@Michael -- thank you for the direction! This task, is proving to be difficult. Mainly, because I realized that I had to put the controls into the block to be removed. I surrounded it by a <div id="notif-$guid"> so that I can remove all of it (rather than having the notification gone, and the controls to delete / mark read sitting there pointing at nothing). <-- that mostly works, because some notifications don't get templated?! (don't get it).

German Michael Zülsdorff Replied 7 months ago

and it doesn't on the fly refresh the all notifications page content.

jquery will be your friend ... you would use something like

$('.anchor-to-be-deleted').remove();

Study OssnComment's delete action + accompanying script and you'll get an idea. :)

cl Hugo Cuellar Replied 7 months ago

Hi Michieal, nice update, all working great.

I added the spanish locale file in the github.

Regards.

us Michieal ~ Coder ~ Replied 7 months ago

V6.1.2.0 ~ Added happy javascript to perform the calls so that the page doesn't have to refresh each time you mark or delete a notification.

Added the Delete All Reactions as a button on the bottom of the Notifications dropdown, next to the See All link.

It's not 100% exactly what I want it to be, but it's very, very close. Ie., it doesn't (yet) have the ability to on the fly refresh just the notifications drop down, and it doesn't on the fly refresh the all notifications page content.
If anyone is willing to help with this, I would be extremely happy!

us Michieal ~ Coder ~ Replied 7 months ago

I would do a pull request, as I can merge it right in.

Because I haven't figured out how to make use of the "friends" on here.

cl Hugo Cuellar Replied 7 months ago

Hi Michiel, how i can send you the locale file? i added you here.

Or i fork your repository and i open a pull request?

us Michieal ~ Coder ~ Replied 7 months ago

I was going off of the comment in the code, that said that the section was to delete the notification as a failsafe, when encountering a broken notification. But, I 100% agree with you that it doesn't remove the notification and you will get directed to a blank page or a 404.

I think, that it should have some kind of automatic notification "cleaner" for when something like that happens and the system has a broken notification... But the "Delete" link in MarkNotifications will do exactly that, albeit not automatically.

Component

Developer: Michieal
License GPL v2
Type: Tools
Requires Ossn Version : 6.1
Latest Version: 6.1.2.0
Last Updated 7 months ago
Repository Url View Repository

Versions