AsyrilSA / log-viewer

MIT License
2 stars 0 forks source link

Sweep: Sweep: Display the timestamp with milliseconds in LogTable and CommunicationLogTable (βœ“ Sandbox Passed) #3

Closed sweep-ai[bot] closed 11 months ago

sweep-ai[bot] commented 11 months ago

PR Feedback: πŸ‘

Description

This pull request includes changes to the following files:

Summary

Fixes #2.


πŸŽ‰ Latest improvements to Sweep:


πŸ’‘ To get Sweep to edit this pull request, you can:

sweep-ai[bot] commented 11 months ago

Sandbox Executions

Ran GitHub Actions for a87523a8e304d764e2dd0e7008b72fce004565e0:

Ran GitHub Actions for d8f5478adcbe9c371de6c72ec61144391f9c799a:

sweep-ai[bot] commented 11 months ago

Rollback Files For Sweep

sweep-ai[bot] commented 11 months ago

Apply Sweep Rules to your PR?

patrickclerc commented 11 months ago

Thanks for the proposal. I prefer to use the local format as the ISO string is less human readable. Could you format the timestamp as the toLocalString() method would do and add the milliseconds ?

sweep-ai[bot] commented 11 months ago

πŸš€ Wrote Changes

Done.

patrickclerc commented 11 months ago

Thanks, is it possible to refactor the format of the timestamp into one unique method, used by the 2 files ?

sweep-ai[bot] commented 11 months ago

πŸš€ Wrote Changes

Done.