-
Notifications
You must be signed in to change notification settings - Fork 418
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
Make Log Timestamp format/timezone configurable #115
Comments
We should assume local and let you change to utc. |
Would require settings storage design |
Could we start with something simple like JSON serializing a settings object to a cookie? |
We could punt this one for preview2 |
The code that configures it runs at startup of the Dashboard, not in the client. We used local storage in the client for other things (e.g. splitter bar size/position) but need something earlier for this. Unless we convert the format at runtime, I guess? But that seems like extra work for no reason. I guess it could go in AppHost's appsettings (and eventually the dashboard's wherever that lives), but if we want a UI for it we need something else I think? |
We should make it part of dashboard launch rather than setting on dashboard. If user want to change the format, restarting the dashboard is fine. We can make it configurable from code where we do AddAspireDashboard or similar in apphost. |
Putting this in the backlog |
Currently the forced timestamps in the project logs are UTC time and of a format that roughly matches the one docker logs use. We should allow users to specify utc vs local and their own format.
The text was updated successfully, but these errors were encountered: