Skip to content
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

Request: Option to include Timestamps for prompts and responses #54

Closed
2 of 3 tasks
bryanschmidty opened this issue Feb 23, 2023 · 8 comments · Fixed by #110
Closed
2 of 3 tasks

Request: Option to include Timestamps for prompts and responses #54

bryanschmidty opened this issue Feb 23, 2023 · 8 comments · Fixed by #110
Labels
enhancement New feature or request

Comments

@bryanschmidty
Copy link

bryanschmidty commented Feb 23, 2023

I noticed that in the xhr reponse for loading a chat conversation, it has a create_time key for the main object, and every child object. It would be neat if we could have a toggle option to "include timestamps" to show on the page, and maybe even include it in the filename (instead of the current timestamp).


Progress Tracking

  • Add timestamp in HTML export
  • Option to toggle timestamp on the response
  • Add create_time variable to filename format
@pionxzh pionxzh added the enhancement New feature or request label Feb 25, 2023
@pionxzh
Copy link
Owner

pionxzh commented Feb 25, 2023

Thanks for the insight. I will check the API you mentioned.

@pionxzh
Copy link
Owner

pionxzh commented Feb 27, 2023

This is the POC that I just implemented.

image
image

@pionxzh
Copy link
Owner

pionxzh commented Feb 27, 2023

@Wolf-SO let's discuss here 😄

pionxzh added a commit that referenced this issue Feb 27, 2023
@pionxzh
Copy link
Owner

pionxzh commented Feb 27, 2023

@kn9 you can contact me via the email

@Wolf-SO
Copy link
Contributor

Wolf-SO commented Feb 27, 2023

@Wolf-SO let's discuss here 😄

As already stated, a locale-aware formatting of the HH:MM part (in the plain text) would be nice. But what you have implemented right now would be a clear improvement.

@pionxzh
Copy link
Owner

pionxzh commented Feb 28, 2023

Based on what we can get from the API, I would like to abandon the hand-written DOM scrapper. We can get the raw data from API without worrying about those edge cases.

@bryanschmidty
Copy link
Author

I love the added timesetamp! Thanks! Looking forward to more updates

@pionxzh
Copy link
Owner

pionxzh commented Apr 7, 2023

You can now use this option to enable the timestamp on page.

image

The {create_time} variable will be traced in #94

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants