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

feat: functionality to view unread messages in chat #2611

Merged
merged 121 commits into from
Dec 28, 2024
Merged

feat: functionality to view unread messages in chat #2611

merged 121 commits into from
Dec 28, 2024

Conversation

disha1202
Copy link
Contributor

@disha1202 disha1202 commented Oct 25, 2024 •
edited by coderabbitai bot
Loading

What kind of change does this PR introduce?

Issue Number:

Fixes #2610

Did you add tests for your changes?

Snapshots/Videos:

If relevant, did you update the documentation?

Summary

Does this PR introduce a breaking change?

Other information

Have you read the contributing guide?

Summary by CodeRabbit

Release Notes

  • New Features

    • Introduced a new field to track unseen messages for users in chats.
    • Added a mutation to allow users to mark chat messages as read, enhancing user interaction.

  • Bug Fixes

    • Improved error handling for non-existent chats and users in message-related operations.

  • Tests

    • Enhanced test coverage for the new unseen messages functionality and the marking of messages as read.

disha1202 and others added 30 commits July 15, 2024 23:08
@disha1202
Updated subscriptions and added mutations for directChat and groupChat
a8c02f8
@disha1202
Update index.ts
fe6907d
@disha1202
Update messageSentToDirectChat.ts
3a9799d
@disha1202
Update createDirectChat.spec.ts
2fa65b1
@disha1202
Update createDirectChat.spec.ts
f68bb01
@disha1202
fix: linting errors
f7089b9
@disha1202
fix: lint errors
2fe5353
@disha1202
fix: formatting issues
7e31c14
@disha1202
Merge branch 'develop' into chat-feature
0bf28a2
@disha1202
fix: formatting issues
9c44ea4
@disha1202
fix: test cases
fa5f6ec
@disha1202
fix test cases
3db8381
@disha1202
Update messageSentToDirectChat.ts
a1379bf
@disha1202
Update messageSentToDirectChat.ts
bd98a6b
added test cases for query directChatById
7ec1773
@disha1202
Update directChatById.spec.ts
3781ede
@disha1202
Update directChatById.spec.ts
8aaf5f3
@disha1202
Update directChatById.spec.ts
e83657d
@disha1202
Added test cases
23a9676
@disha1202
fix: test cases
9576d4a
@disha1202
fix: tests
c2af22d
@disha1202
fix test cases
ad48619
@disha1202
Merge branch 'chat-feature' of https://github.com/disha1202/talawa-api …
77c7fde 
…into chat-feature
@disha1202
added support to reply to direct chat and group chat
d18843a
@disha1202
Merge branch 'develop' of https://github.com/disha1202/talawa-api int…
cf27ca6 
…o reply-functionality
@disha1202
fix: test cases
8469653
@disha1202
updated schema for chat
87ad70f
@disha1202
removed console logs
f936adc
@disha1202
Merge branch 'develop' into reply-functionality
babf1c0
@disha1202
Merge branch 'develop' into reply-functionality
c84f33a
@github-actions GitHub Actions
Copy link

This pull request did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please verify it has no conflicts with the develop branch and rebase if needed. Mention it now if you need help or give permission to other people to finish your work.

@github-actions github-actions bot added no-pr-activity No pull request activity and removed no-pr-activity No pull request activity labels Nov 21, 2024
@palisadoes
Copy link
Contributor

@disha1202 Are you still working on this?

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 4, 2024

This pull request did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please verify it has no conflicts with the develop branch and rebase if needed. Mention it now if you need help or give permission to other people to finish your work.

@github-actions github-actions bot added no-pr-activity No pull request activity and removed no-pr-activity No pull request activity labels Dec 4, 2024
@Cioppolo14
Copy link
Contributor

Closing due to inactivity.

@Cioppolo14 Cioppolo14 closed this Dec 13, 2024
@palisadoes
Copy link
Contributor

Reopening. GSoC

@palisadoes palisadoes reopened this Dec 13, 2024
@palisadoes
Copy link
Contributor

@disha1202 Please make the required changes. We'd like to get this feature added.

@palisadoes
Copy link
Contributor

  1. Please fix the failing tests. Click on the Details links for more information.
  2. Please make sure that CodeRabbit.ai approves your changes

@disha1202
Copy link
Contributor Author

Hi @palisadoes
The typecheck test is failing which is not related to the changes in this PR and is fixed in
#2770

@palisadoes palisadoes added the ignore-sensitive-files-pr The contributor has a legitimate reason for editiing protected files label Dec 20, 2024
@palisadoes
Copy link
Contributor

@disha1202 The PR was merged and the failed tests persist

@palisadoes
Copy link
Contributor

@disha1202 Can we merge this now? Can you make changes so that coderabbit.ai approves?

@disha1202 disha1202 mentioned this pull request Dec 28, 2024
Merged
@palisadoes palisadoes merged commit 8691a24 into PalisadoesFoundation:develop Dec 28, 2024
12 checks passed
This was referenced Dec 28, 2024
Closed
Closed
Merged
@coderabbitai coderabbitai bot mentioned this pull request Jan 28, 2025
Merged
6 tasks
@coderabbitai coderabbitai bot mentioned this pull request Mar 8, 2025
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@palisadoes palisadoes palisadoes approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees
No one assigned
Labels
GSoC Priority ignore-sensitive-files-pr The contributor has a legitimate reason for editiing protected files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@disha1202 @palisadoes @Cioppolo14