FIX: correctly respects full name settings in channel title (#22566)

This commit is contained in:
Joffrey JAFFEUX 2023-07-12 16:59:47 +02:00 committed by GitHub
parent fbe0e4c78c
commit aa9c7dc924
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 3 deletions

View File

@ -19,7 +19,9 @@
<span class="chat-channel-title__name">{{this.usernames}}</span>
{{else}}
{{#let @channel.chatable.users.firstObject as |user|}}
<span class="chat-channel-title__name">{{user.username}}</span>
<span class="chat-channel-title__name">
<ChatUserDisplayName @user={{user}} />
</span>
{{#if this.showUserStatus}}
<UserStatusMessage
@class="chat-channel-title__user-status-message"
@ -35,8 +37,6 @@
/>
{{/let}}
{{/if}}
{{else}}
<span class="chat-channel-title__name">Add users</span>
{{/if}}
</div>
</div>

View File

@ -80,6 +80,24 @@ module("Discourse Chat | Component | chat-channel-title", function (hooks) {
);
});
test("direct message channel - one user showing full name", async function (assert) {
this.siteSettings.prioritize_username_in_ux = true;
this.user = fabricators.user({
username: "joffrey",
name: "Joffrey Baratheon",
});
this.channel = fabricators.directMessageChannel({
chatable: fabricators.directMessage({ users: [this.user] }),
});
await render(hbs`<ChatChannelTitle @channel={{this.channel}} />`);
assert
.dom(".chat-user-display-name__username.-first")
.hasText(this.user.username);
assert.dom(".chat-user-display-name__name").hasText(this.user.name);
});
test("direct message channel - multiple users", async function (assert) {
const channel = fabricators.directMessageChannel();