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.

Sign up for GitHub

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

Jump to bottom

[🐴] don't include blocked convos in unread count #4082

Merged
merged 5 commits into from
May 17, 2024
Merged

[🐴] don't include blocked convos in unread count #4082

merged 5 commits into from
May 17, 2024

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented May 17, 2024 •
edited by estrattonbailey
Loading

Unblocked/blocked
CleanShot 2024-05-17 at 14 34 56@2x
CleanShot 2024-05-17 at 14 34 47@2x

@render Render
Copy link

render bot commented May 17, 2024

Your Render PR Server URL is https://social-app-pr-4082.onrender.com.

Follow its progress at https://dashboard.render.com/web/srv-cp3ls66ct0pc73em0leg.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 17, 2024 •
edited
Loading

Old size New size Diff
6.82 MB 6.82 MB 15 B (0.00%)

estrattonbailey
estrattonbailey approved these changes May 17, 2024
View reviewed changes
src/state/queries/messages/list-converations.ts Outdated
member => member.did !== currentAccount?.did,
)
const blocked =
otherMember?.viewer?.blockedBy || otherMember?.viewer?.blocking
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should also check for blockedByList here. I think that's it, but peep the types for ViewerState just in case

@estrattonbailey
Use moderateProfile
29b2987
@estrattonbailey
Merge remote-tracking branch 'origin/main' into samuel/ignore-blocked
8a17a99 
* origin/main:
  [🐴] NUX (#4062)
  [🐴] Reduce header size (#4078)
  [🐴] Don't always show notification for everything (#4083)
  [🐴] Additional tweaks to the message list (#4075)
@estrattonbailey
Handle blocked state in chat list
cb35174
@estrattonbailey
Fix logic formatting, add todo
b7a40d2
@estrattonbailey estrattonbailey merged commit f42f7fa into main May 17, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@estrattonbailey estrattonbailey estrattonbailey approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mozzius @estrattonbailey