-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updating normalizerConfig
after initial render
#38
Comments
I think we could do this. I wanted to be sure though, is it necessary for you only in development? Because like mentioned at https://github.com/klis87/normy/tree/master?tab=readme-ov-file#debugging-arrow_up , typically this is not meant to be used in production |
Ah I did miss that line;
I was interested in logging in Prod for Admins only. But it's not a super important feature for me. |
In theory we could also make it configurable to allow it on production. However, I wonder if this did not spam your logs on production :) It would cause tons of logs. Anyway, this feature was exposed mostly as curiocity, so that developers could see what is really done by Normy under the hood, and... to debug some problems if they arise etc. |
These logs have extremely helpful.
No worries if you'd like to leave as is. Always a tradeoff of complexity. |
is your setup broken in production if you use NODE_ENV !== 'production' ? If it is ok, then you would have those logs in production. And epxosing update of this config dynamically would be very easy then. |
It would be awesome if there was a way to update the config
normalizerConfig
specifically for my current needsdevLogging
after the initial render.I'd like to expose this in an admin panel so admins can turn on logging if needed.
I'm currently using the react-query variant of Normy if it makes a difference.
The text was updated successfully, but these errors were encountered: