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.

Sign up for GitHub

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

Jump to bottom

feat: expose sent at for messages #1140

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

feat: expose sent at for messages #1140

wants to merge 23 commits into from

Conversation

SputNikPlop
Copy link
Collaborator

This lets the user see on the long menu about how long it was from when the last user in chat sent a message.

SputNikPlop
SputNikPlop commented Jan 17, 2024
View reviewed changes
lib/l10n/app_de.arb Outdated Show resolved Hide resolved
@kevmo314
Copy link
Contributor

Screenshot for the behavior?

@SputNikPlop
Copy link
Collaborator Author

@kevmo314 This is the current behavior, I don't think I quite understand the part of hours ago etc
IMG_2186 Medium

@SputNikPlop
Copy link
Collaborator Author

@kevmo314 Any other feedback on this one? I still am unsure how you get the plurals correct Sent 1 minutes ago ago should be Sent 1 minute ago

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juaoose juaoose Awaiting requested review from juaoose

@chungwwei chungwwei Awaiting requested review from chungwwei

@kevmo314 kevmo314 Awaiting requested review from kevmo314

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SputNikPlop @kevmo314