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

Fix timestamp bug #2

Merged
merged 3 commits into from
Dec 11, 2024
Merged

Fix timestamp bug #2

merged 3 commits into from
Dec 11, 2024

Conversation

shatfield4
Copy link
Collaborator

Fixed a bug where messages that were just sent in the embed widget were not using the correct sentAt time.

  • Any messages that were sent and appended to the history array did not have a sentAt attribute causing the time to fallback on the current time which would rerender on key press of the chat input box
  • Fixed by storing current time message was sent at and modified the chat util to preserve this time so that it stays accurate and does not fallback anymore on the current time

Copy link
Member

@timothycarambat timothycarambat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets improve the data parser for handling null edge cases and then bump with those changes Mintplex-Labs/anything-llm#2806

@timothycarambat timothycarambat merged commit 4a83ad6 into main Dec 11, 2024
@timothycarambat timothycarambat deleted the fix-timestamp-bug branch December 11, 2024 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants