Skip to content

Commit

Permalink
FIX: empty state message on the group messages pages (#14371)
Browse files Browse the repository at this point in the history
  • Loading branch information
AndrewPrigorshnev committed Sep 20, 2021
1 parent a842ea8 commit b6ccc9c
Showing 1 changed file with 18 additions and 2 deletions.
@@ -1,5 +1,5 @@
import I18n from "I18n";
import createPMRoute from "discourse/routes/build-private-messages-route";
import I18n from "I18n";
import { findOrResetCachedTopicList } from "discourse/lib/cached-topic-list";

export default (inboxType, filter) => {
Expand Down Expand Up @@ -35,7 +35,16 @@ export default (inboxType, filter) => {

return lastTopicList
? lastTopicList
: this.store.findFiltered("topicList", { filter: topicListFilter });
: this.store
.findFiltered("topicList", { filter: topicListFilter })
.then((topicList) => {
// andrei: we agreed that this is an anti pattern,
// it's better to avoid mutating a rest model like this
// this place we'll be refactored later
// see https://github.com/discourse/discourse/pull/14313#discussion_r708784704
topicList.set("emptyState", this.emptyState());
return topicList;
});
},

afterModel(model) {
Expand Down Expand Up @@ -69,6 +78,13 @@ export default (inboxType, filter) => {
this.controllerFor("user-private-messages").set("group", this.group);
},

emptyState() {
return {
title: I18n.t("user.no_messages_title"),
body: "",
};
},

dismissReadOptions() {
return {
group_name: this.get("groupName"),
Expand Down

0 comments on commit b6ccc9c

Please sign in to comment.