Skip to content
This repository has been archived by the owner on Apr 16, 2023. It is now read-only.

[🚀] Custom event when significant client properties change #95

Open
MarkL4YG opened this issue Sep 3, 2019 · 0 comments · May be fixed by #96
Open

[🚀] Custom event when significant client properties change #95

MarkL4YG opened this issue Sep 3, 2019 · 0 comments · May be fixed by #96
Labels
api Related to API changes / additions (internal API) enhancement New feature or request performance Performance impacting stuff service Issues & PRs related to a specific service.

Comments

@MarkL4YG
Copy link
Member

MarkL4YG commented Sep 3, 2019

Describe the solution you'd like
Nearly all property changes on users do not trigger events which makes detecting them rather hard to detect. The framework should provide a way for plugins to detect changes and act upon them precisely - instead of crawling over all clients each time. This would be similar to the channeledited event which already offers a map of changed properties.

The implementation approach could also feature some sort of change handler that could be registered on a property.
Lastly, this could also be used for channels or basically all data holders in general.

Describe why you would like to see this implemented
A reduction in boilerplate code would be the immediate result as all plugins that work with changing client properties will no longer have to crawl over all/watched channels after a cache-update.

Additional context
Feature requested by @genius42 with regard to being able to detect changes in assigned server groups.

@MarkL4YG MarkL4YG added enhancement New feature or request service Issues & PRs related to a specific service. api Related to API changes / additions (internal API) performance Performance impacting stuff labels Sep 3, 2019
@MarkL4YG MarkL4YG added this to the Stable Release 1.1.0 milestone Sep 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api Related to API changes / additions (internal API) enhancement New feature or request performance Performance impacting stuff service Issues & PRs related to a specific service.
Development

Successfully merging a pull request may close this issue.

1 participant