search

agateblue / django-dynamic-preferences

Dynamic global and instance settings for your django project
https://django-dynamic-preferences.readthedocs.org/
BSD 3-Clause "New" or "Revised" License
350 stars 87 forks source link

Fix #310 The `preference_updated` signal is now triggered when using the REST API #311

Open KrYpTeD974 opened 1 week ago

KrYpTeD974 commented 1 week ago

Changes have been made to the update method of the PreferenceSerializer to trigger the preference_updated signal.

It has been tested for single updates and bulk updates.

I have also added the preference instance to the signal's parameters as it may be needed when you want to retrieve the model bound to a preference, like a workspace, or a user.

KrYpTeD974 commented 1 week ago

@agateblue Sorry to ask, but did you review this PR ? Don't hesitate to ask for any modification!

agateblue commented 2 days ago

Hi @KrYpTeD974 , I'm very sorry for the delay. Life has been tough lately, I haven't forgotten you and this will be reviewed, but it might take a couple more weeks. Thank you for your patience.

KrYpTeD974 commented 1 day ago

No worries, take the time you need, life can be tough but time generally help a lot.