xh / hoist-react

🏗️ ⚛️ The XH Hoist toolkit for React
https://xh.io
Apache License 2.0
24 stars 9 forks source link

Admin log levels configuration grid does not refresh after update #3800

Closed amcclain closed 1 month ago

amcclain commented 1 month ago

When adding or updating a log level, the entire grid should refresh after any individual update, to reflect potential changes to effective logging levels on other rows.

Eg I have log level rows for com.myapp and com.myapp.pkg. Both start with no override showing default INFO level. I update parent com.myapp to debug. After update com.myapp.pkg should also show effective level of debug.

If after making that update I close the log levels dialog and re-open, I see the correct effective levels across all rows.

lbwexler commented 1 month ago

Yes, of course, agree that really is important and would really aid in understanding how this works.

Looked at it, and weirdly we don't have any great hooks for this in RestGridModel.

lbwexler commented 1 month ago

actual found a little workaround see PR -- think making RestGridModel more flexible to handle this is a bigger project, but felt it was worth it to get this fix in ASAP