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

feature: Add support for making messages read/unread. #689

Draft
wants to merge 9 commits into
base: dev
Choose a base branch
from

Conversation

shirblc
Copy link
Contributor

@shirblc shirblc commented Nov 1, 2024

Description

A clear and concise description of what the bug or the new feature is.

Issue link

If there is one, add a link to the open issue referencing this bug.

Expected behavior

For bugs: A clear and concise description of what you expected to happen.

Your solution

A clear and concise description of what you did to fix the bug or implement the new feature.

Additional information

Add any other information about the problem / feature request here.

Copy link

codecov bot commented Nov 1, 2024

Codecov Report

Attention: Patch coverage is 57.14286% with 12 lines in your changes missing coverage. Please review.

Project coverage is 95.72%. Comparing base (8e7400d) to head (50a0457).

Files with missing lines Patch % Lines
controllers/messages.py 36.84% 12 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #689      +/-   ##
==========================================
- Coverage   96.60%   95.72%   -0.89%     
==========================================
  Files          27       27              
  Lines        1238     1263      +25     
==========================================
+ Hits         1196     1209      +13     
- Misses         42       54      +12     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

1 participant