ring_logger icon indicating copy to clipboard operation
ring_logger copied to clipboard

Addition of New Command for Date Entry Navigation

Open Shivaji-Dhepale opened this issue 1 year ago • 4 comments

This PR introduces goto date entry navigation to the Ringlogger viewer command system:

Command string can be as follows:

  1. Ringlogger.viewer("d 2025-12-11 10:20:11")

Addition of New Command for Date Entry Navigation: A new command has been added to facilitate easy navigation to the date entry section. This improves the overall user experience when working with time-sensitive data.

Testing:

Unit Test cases are updated for these enhancements Conducted manual testing to validate the functionality of the newly introduced command, ensuring it work as expected in various scenarios. Confirmed that the new date entry navigation command accurately redirects to the date entry section, enhancing usability for time-sensitive operations

Demo:

Command FAQM-1584-goto-date-command

Response FAQM-1584-goto-date-test-preview

Shivaji-Dhepale avatar Feb 14 '25 06:02 Shivaji-Dhepale

Hi @fhunleth , addressed your review comments. Please let us know if anything else required.

Shivaji-Dhepale avatar Feb 21 '25 08:02 Shivaji-Dhepale

Hi @fhunleth , As requested, I've created a separate PR for the date entry navigation command. Additionally, following your suggestion, I’ve provided ISO8601-compliant strings for start_time and duration for Elixir 1.17 and above. We have ensured that the Duration module loads first before calling any other related functions in case of Elixir lower versions(below 1.17).

Shivaji-Dhepale avatar Feb 24 '25 10:02 Shivaji-Dhepale

Hello @fhunleth, We're excited to hear your suggestions and look forward to your input!

Shivaji-Dhepale avatar Feb 28 '25 04:02 Shivaji-Dhepale

Hi @fhunleth, image This test failure appears to be happening randomly and only for one of the Elixir base images. The difference in assertion results suggests a potential issue with ANSI escape codes or encoding inconsistencies. Could you please retrigger the pipeline to see if this is a transient issue?

Shivaji-Dhepale avatar Mar 31 '25 12:03 Shivaji-Dhepale

Hello @fhunleth , we are excited to hear your suggestions and looking forward to your inputs!

Shivaji-Dhepale avatar Apr 10 '25 12:04 Shivaji-Dhepale

Thanks! Sorry for the very long delay with reviewing this.

fhunleth avatar Apr 11 '25 00:04 fhunleth