Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(badge): show badge when mentioned even if channel is muted #8724

Merged
merged 1 commit into from
Dec 12, 2022

Conversation

jrainville
Copy link
Member

Fixes #8723

@jrainville jrainville requested review from a team, dlipicar and mprakhov and removed request for a team December 9, 2022 16:42
@status-im-auto
Copy link
Member

status-im-auto commented Dec 9, 2022

Jenkins Builds

Click to see older builds (6)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 88ba1f0 #1 2022-12-09 16:48:17 ~5 min imports 📄log
✔️ 88ba1f0 #1 2022-12-09 16:48:24 ~5 min linux-cpp 📄log
✔️ 88ba1f0 #1 2022-12-09 16:52:59 ~9 min macos 🍎dmg
✔️ 88ba1f0 #1 2022-12-09 16:53:14 ~10 min linux 📦tgz
✔️ 88ba1f0 #1 2022-12-09 17:07:48 ~24 min windows 💿exe
✔️ 88ba1f0 #1 2022-12-09 17:11:29 ~28 min linux-e2e 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 5a153d7 #2 2022-12-09 19:04:07 ~4 min imports 📄log
✔️ 5a153d7 #2 2022-12-09 19:04:46 ~4 min linux-cpp 📄log
✔️ 5a153d7 #2 2022-12-09 19:06:08 ~6 min macos 🍎dmg
✔️ 5a153d7 #2 2022-12-09 19:13:07 ~13 min linux 📦tgz
✔️ 5a153d7 #2 2022-12-09 19:18:39 ~18 min windows 💿exe
✔️ 5a153d7 #2 2022-12-09 19:25:28 ~25 min linux-e2e 📄log
✔️ 0367f04 #3 2022-12-12 16:12:31 ~7 min imports 📄log
✔️ 0367f04 #3 2022-12-12 16:12:56 ~7 min linux-cpp 📄log
✔️ 0367f04 #3 2022-12-12 16:24:33 ~19 min linux 📦tgz
✔️ 0367f04 #3 2022-12-12 16:29:28 ~24 min windows 💿exe
✔️ 0367f04 #3 2022-12-12 16:32:35 ~27 min linux-e2e 📄log
✔️ 0367f04 #3 2022-12-12 16:33:22 ~28 min macos 🍎dmg

@jrainville
Copy link
Member Author

I messed up my rebase, do not merge

@jrainville
Copy link
Member Author

Turns out my other branch was on top of this, but an older commit. So I merged this by accident, but anyway, it was already reviewed here, so I'll just merge this too to fix the comment

@jrainville jrainville merged commit 2b9c1f9 into master Dec 12, 2022
@jrainville jrainville deleted the fix/community-muted-badge branch December 12, 2022 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Muted communities don't produce a badge when mentioned
5 participants