Skip to content

Conversation

Aastha-1411
Copy link
Contributor

@Aastha-1411 Aastha-1411 commented Jul 22, 2025

Closes: #7
This PR addresses the issue with incorrect timestamp display by ensuring the time shown is based on the user's local time, not the server’s. The app now correctly renders local timestamps both locally and after deployment.

Changes Made:
Replaced the old timestamp logic with one that considers the user's local timezone using st.context.timezone_offset. This ensures that the time displayed is accurate for the end-user, not just the server's timezone (e.g., UTC on Streamlit Cloud).

I verified the change by deploying the project myself on community cloud to check if .
it is making chnages as per the given time. You can see the image below.

Screenshot 2025-07-22 190557

I was chatting with the bot on 7:04 PM only. So its working fine.

@eccentriccoder01
Copy link
Owner

That's great, @Aastha-1411 you've done a great job❕But:

  1. The convo delete button doesn't appear in your screenshot, which means you haven't pulled the latest changes. Pull it at first, as one PR has recently been merged.
  2. After doing that, link this PR to the issue which you created. It is unlinked.

@Aastha-1411 Aastha-1411 changed the title Fix Timestamp issue in chat interface Fix Timestamp issue in chat interface Closes: #7 Jul 22, 2025
@Aastha-1411
Copy link
Contributor Author

That's great, @Aastha-1411 you've done a great job❕But:

  1. The convo delete button doesn't appear in your screenshot, which means you haven't pulled the latest changes. Pull it at first, as one PR has recently been merged.
  2. After doing that, link this PR to the issue which you created. It is unlinked.

I have pulled the latest changes and linked my PR to the issue. You can now merge my PR.

@Aastha-1411 Aastha-1411 changed the title Fix Timestamp issue in chat interface Closes: #7 Fix Timestamp issue in chat interface Jul 22, 2025
@eccentriccoder01
Copy link
Owner

eccentriccoder01 commented Jul 22, 2025

I've checked this deployment link of yours: talkheal-kfiantnh6xkhtd3tzbg5bd.streamlit.app
And it doesn't show the delete buttons beside the conversations.

image

Whilst the latest changes will have this:
image

It maybe that your pulled changes weren't resolved properly, or maybe they were omitted totally after pulling, or maybe some logical error is hindering the delete functionality.

I hope you now understand. Correct me if I'm wrong about your deployment link (if you have any other latest deployment).

@Aastha-1411
Copy link
Contributor Author

Screenshot 2025-07-23 085403

Now the changes are there, you can check the link too : https://talkheal-kfiantnh6xkhtd3tzbg5bd.streamlit.app/

Thanks for your time and patience.

I've checked this deployment link of yours: talkheal-kfiantnh6xkhtd3tzbg5bd.streamlit.app And it doesn't show the delete buttons beside the conversations.

image Whilst the latest changes will have this: image

It maybe that your pulled changes weren't resolved properly, or maybe they were omitted totally after pulling, or maybe some logical error is hindering the delete functionality.

I hope you now understand. Correct me if I'm wrong about your deployment link (if you have any other latest deployment).

Screenshot 2025-07-23 085403

Now you can see the delete option. The deployed link is here: https://talkheal-kfiantnh6xkhtd3tzbg5bd.streamlit.app/

Thanks for your time and patience.

@eccentriccoder01
Copy link
Owner

@Aastha-1411 Happy that the issue is successfully resolved, great job, thanks❕

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

Successfully merging this pull request may close these issues.

Message timestamps don’t reflect real/local time
2 participants