Skip to content

Feature: admin notifications#1352

Open
SpeedyD wants to merge 16 commits intolk-arpg:developfrom
SpeedyD:feature/admin-notifications
Open

Feature: admin notifications#1352
SpeedyD wants to merge 16 commits intolk-arpg:developfrom
SpeedyD:feature/admin-notifications

Conversation

@SpeedyD
Copy link
Copy Markdown
Contributor

@SpeedyD SpeedyD commented Nov 1, 2025

Drafting cause still in progress-

Got @preimpression's permission to port this, and so far that is purely that. Still gonna look coding through to see if things are necessary.. like, I think "UserCharacterLog" is not necessary for this? Gonna have to test that..

Also, need to still add toggles for users and overall, since this is somewhat obtrusive.

Also thinking of just making the circles badges? I think the huge circle is a bit much and bootstrap's badges are perfect for this.. Opinions on that welcome.

preimpression and others added 8 commits August 1, 2020 22:40
…ts the number of items in the queue. Staff with "manage_masterlist" will get to see trades, transfers, and design queues. Staff with "manage_submissions" will see claim and prompt queue.
…ion' into feature/admin-notifications

* preimpression/extension/admin-notification:
  refactor: Adjusts way User model collects admin notifications and adds Reports to the notifications.
  Adds an admin notification bubble (similar to on deviantart) that lists the number of items in the queue. Staff with "manage_masterlist" will get to see trades, transfers, and design queues. Staff with "manage_submissions" will see claim and prompt queue.
@itinerare itinerare added the enhancement New feature or request label Nov 2, 2025
@SpeedyD SpeedyD marked this pull request as ready for review January 19, 2026 21:33
@SpeedyD
Copy link
Copy Markdown
Contributor Author

SpeedyD commented Jan 19, 2026

I've opted to do other changes later. Currently I'm just.. This needs to get out. I don't know why I left it hanging.

@itinerare
Copy link
Copy Markdown
Member

This should have toggles before being merged, imo.

@SpeedyD
Copy link
Copy Markdown
Contributor Author

SpeedyD commented Mar 3, 2026

Okay, so, updates.

  1. In the extensions.php config file, the entirety of Admin Notifications can be disabled. It's enabled by default.
  2. An admin can, for themselves, enable/disable the number, or make it slightly more compact.
  3. I've added a check if classes exist for Affiliates and the Donation Shop- both extensions still require to be added as 'use' case in the User model, but at least the notification side should be automated now.

I hope this is satisfactory. I've been intending on doing this since I stated 'draft for now' in the intro message, but simply never got to them.. until now.

@itinerare itinerare added needs review Pull requests that are pending community review has:migration labels Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request has:migration needs review Pull requests that are pending community review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants