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

unreads [nfc]: Update dartdoc following stream/topic-muting logic #624

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

chrisbobbe
Copy link
Collaborator

In #423 "Implement stream and topic muting", the Unreads class did start doing some filtering based in stream/topic muting, specifically in these methods:

  • countInStream
  • countInAllMessagesNarrow

So, update the dartdoc so it doesn't claim that this filtering is an entirely separate concern.

@chrisbobbe chrisbobbe requested a review from gnprice April 17, 2024 22:24
In zulip#423 "Implement stream and topic muting", the Unreads class did
start doing some filtering based in stream/topic muting,
specifically in these methods:
- countInStream
- countInAllMessagesNarrow

So, update the dartdoc so it doesn't claim that this filtering is an
entirely separate concern.
@gnprice
Copy link
Member

gnprice commented Apr 26, 2024

Sure, looks good; merging.

@gnprice gnprice force-pushed the pr-unreads-dartdoc-muting branch from 4e9da09 to fcd74e6 Compare April 26, 2024 21:56
@gnprice gnprice merged commit fcd74e6 into zulip:main Apr 26, 2024
1 check passed
@chrisbobbe chrisbobbe deleted the pr-unreads-dartdoc-muting branch April 26, 2024 21:58
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.

2 participants