Michieal ~ Coder ~
2 years 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.
Michieal ~ Coder ~
Replied 2 years ago
New release uploaded complete with the suggested changes.
Hugo, if you would be so kind, I would genuinely like it if you would give the new version a test spin!
Again thank you Hugo, Michael, and Arsalan! I appreciate the help!
Michieal ~ Coder ~
Replied 2 years ago
I see! Okay, Yeah uhm, I don't have users with special characters like umlauts in their names yet. Sorry for not fully understanding the issue, Hugo! Thank you for finding it!!
Also, thank you Arsalan and Michael for your help with this!!!
I will post a new version here in a few minutes, after adding in your suggestions to the the merged code.
Arsalan Shah
Replied 2 years ago
Yes agree
Michael Zülsdorff
Replied 2 years ago
Yup, and it makes sense to remove it in the end like
return str_replace('<?xml encoding="UTF-8">', '', $doc->saveHTML());
Otherwise you'll get a lot of unnecessary lines in your output
Arsalan Shah
Replied 2 years ago
This is common issue when using DOMDocuments the fix is sent to https://github.com/michieal/marknotification/pull/1
Michael Zülsdorff
Replied 2 years ago
Ah ok, got it.
Re UTF-8 issue: I can confirm Hugo's finding. After renaming my beloved testuser Honey Bunny to Höney Bünny the trouble starts. And actually only on those notification types which are hooked by your component.
Michieal ~ Coder ~
Replied 2 years ago
@Michael - Users starting using the thumbs up (Like) for comments for them to keep track of what they have read. So, 99% of the notifications I received were like reactions on comments, in an active topic/post. Hence, the button to clean up some notifications, as a convenience.
Erassus ︎ ︎
Replied 2 years ago
Component Disabled:
Component Enabled:
Name for testing:
➳ Kᴀᴛᴇ Bɪsʜᴏᴘ ©
Michael Zülsdorff
Replied 2 years ago
my users started to use "Like" reactions to mark things as read.
Sorry, but I do not understand this.
Likening a post results in a new notification in the author's account, yes, but it doesn't "mark" that notification in any way
Michieal ~ Coder ~
Replied 2 years ago
6.1.1.0 -- For this version, delete the component before installing this version to clean up any lingering files.
New in this version:
I added in a "Delete All Reactions" button to the Notifications/All page. This is because after I added in the modifications that @Michael made to the posts that increased interactions, my users started to use "Like" reactions to mark things as read. I ended up with ~70 notifications in an hour! This is my answer to allow users to clean up non-essential notifications.
Additionally, cleaned up the files; made minor css changes, etc. Tested to make sure that things worked on my live site and dev machine. Verified that the code does not alter user names.
