You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
def handle_save_message(self, json_path):
if json_path == "":
json_path = "messages.json"
if not json_path.endswith(".json"):
json_path += ".json"
with open(json_path, 'w') as f:
json.dump(self.messages, f, indent=2)
display_markdown_message(f"> messages json export to {os.path.abspath(json_path)}")
Right now, the messages overwrite by default when no path is provided. That shouldn't be the case. You could lose conversations by mistake - especially without warnings in place.
Describe the solution you'd like
I'd prefer that the saved by date, and an incremental suffix.
Describe alternatives you've considered
Considered just added a warning to prevent overwriting but that is a lot of friction for every time you want to save quickly. The best user experience is storing a new json, especially since they aren't large files it's fine. We can even add a simple cleanup command if needed.
The warning can still be added but this will only be when you type in the same file name.
Additional context
No response
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
Right now, the messages overwrite by default when no path is provided. That shouldn't be the case. You could lose conversations by mistake - especially without warnings in place.
Describe the solution you'd like
I'd prefer that the saved by date, and an incremental suffix.
Describe alternatives you've considered
Considered just added a warning to prevent overwriting but that is a lot of friction for every time you want to save quickly. The best user experience is storing a new json, especially since they aren't large files it's fine. We can even add a simple cleanup command if needed.
The warning can still be added but this will only be when you type in the same file name.
Additional context
No response
The text was updated successfully, but these errors were encountered: