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

Render ignored users setting regardless of if there are any #5860

Merged
merged 1 commit into from
Apr 28, 2021
Merged

Render ignored users setting regardless of if there are any #5860

merged 1 commit into from
Apr 28, 2021

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

@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

List of ignored users is only displayed if there are actually ignored users
3 participants