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

Add empty threads info texts and update translations files. #5997

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ShoaibIqbal452
Copy link

Ticker Link: N/A

Context:

While playing around with chats I noticed I had no threads and when I navigated to threads page it seemed all empty and It took me a second to figure out it was all empty. It is not a bug but I felt it would be a good in context of a user to have info/message indicating the threads page is empty.

image

Changes:

1: Check for empty threads in Threads.svelte.
2: Add a div to display empty threads info incase threads are empty.
3: Declare translation strings in plugin.
4: Update all translation files using google translate.

image

Impact:

If there are no threads empty threads page displays infotext now which improved user experience.

Steps to reproduce

Create a new workspace with no users. Navigate to chat and then to threads page.

Copy link

typo-app bot commented Jul 3, 2024

Typo Code Review 📊

✅ All checks passed!

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

Successfully merging this pull request may close these issues.

2 participants