Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Conversation

samhh
Copy link
Contributor

@samhh samhh commented Apr 13, 2021

@SimonBrandner
Copy link
Contributor

@samhh, thanks for working on this! Could you please include a sign-off as described here?

@samhh
Copy link
Contributor Author

samhh commented Apr 13, 2021

I knew I'd forgotten something! 🙂 I've rebased with commit sign-off. First time I've used that Git feature!

Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

can we also get a screenshot please to show what it looks like in context? Thanks!

const userIds = ignoredUserIds
.map((u) => <IgnoredUser
const userIds = !ignoredUserIds?.length
? 'You have no ignored users.'
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be a translated string (_t and yarn i18n)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made the change. I've amended the commit as it's still a very small diff.

@samhh
Copy link
Contributor Author

samhh commented Apr 13, 2021

2021-04-13-214216_826x379_scrot

Signed-off-by: Sam A. Horvath-Hunt <hello@samhh.com>
@jryans jryans requested a review from turt2live April 23, 2021 12:44
@turt2live turt2live merged commit 3890bec into matrix-org:develop Apr 28, 2021
@samhh samhh deleted the always-render-ignored-users branch April 28, 2021 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants