Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DEV: Don't fetch tagNotifications when additional tags are present (#23351) 

We don't display them, see: 006a5166e5/app/assets/javascripts/discourse/app/components/d-navigation.js (L55-L55)
This commit is contained in:
Jarek Radosz 2023-08-31 16:40:33 +02:00 committed by GitHub
parent 006a5166e5
commit 06a9963cbb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/assets/javascripts/discourse/app/routes/tag-show.js
View File

@ -59,7 +59,7 @@ export default DiscourseRoute.extend({
const filterType = this.navMode.split("/")[0];
let tagNotification;
if (tag && tag.id !== NONE && this.currentUser) {
if (tag && tag.id !== NONE && this.currentUser && !additionalTags) {
// If logged in, we should get the tag's user settings
tagNotification = await this.store.find(
"tagNotification",

5
app/assets/javascripts/discourse/tests/acceptance/tags-intersection-test.js
View File

@ -12,11 +12,6 @@ acceptance("Tags intersection", function (needs) {
needs.settings({ tagging_enabled: true });
needs.pretender((server, helper) => {
server.get("/tag/first/notifications", () => {
return helper.response({
tag_notification: { id: "first", notification_level: 1 },
});
});
server.get("/tags/intersection/first/second.json", () => {
return helper.response({
users: [],