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

Wording / Translation: "[channel] is live streaming in [video]" Notification #6803

Closed
vencabot opened this issue Jan 2, 2025 · 2 comments
Closed

Comments

@vencabot
Copy link

vencabot commented Jan 2, 2025

Describe the problem to be solved

I visited my PeerTube page today and got the following notification:

DannekRose Live is live streaming in [EN] Quick Test

It was from 4 hours ago. I clicked the link and, of course, the stream has been over for a long time.

This is a bit misleading, because the notification uses present-tense language. I understand that this is because the notification message was composed 4 hours ago and it doesn't update, but perhaps this could be handled better.

Describe the solution you would like

I think it would be less confusing if the notification were worded along the lines of

DannekRose Live went live: [EN] Quick Test

This would also address the subtle awkwardness of the 'in' particle, since many users may not understand how PeerTube streams go live 'into' a particular pre-configured video space.

If I see that DannekRose Live IS live, as of 4 hours ago, I presume that his stream has been going on for 4 hours and that it's live right now.

If I see that DannekRose Live WENT live as of 4 hours ago, I make no such presumptions. Maybe he's still streaming, maybe not.

Anyway, this is an extremely minor issue, but I thought I'd bring it up. Thank you for everything that you do for this project.

@vencabot vencabot changed the title Minor UI Issue: "[channel] is live streaming in [video]" Notification Wording / Translation: "[channel] is live streaming in [video]" Notification Jan 2, 2025
@Chocobozzz
Copy link
Owner

Very nice suggestion! Should be implemented in 82246a0

@vencabot
Copy link
Author

Thank you so much, sir!

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

No branches or pull requests

2 participants