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

feat: Attempted to complete read tick feature for NodeBB group chats.… #77

Closed
wants to merge 1 commit into from

Conversation

khiyami
Copy link

@khiyami khiyami commented Oct 19, 2024

Pull Request Description

Overview

This PR implements the initial attempt at the blue tick feature for group chat messages in NodeBB, as described in issue #20. The objective was to change the tick color to blue when all users in a group view the message. While the feature is partially implemented, it remains non-functional despite several efforts and attempted methods.


What Was Done

  • I successfully added a tick element to group chat messages.
  • I tried to make the tick turn blue when viewed by all members of the group, but the implementation did not fully succeed.
  • The following approaches were attempted:
    1. Tracking users: Added logic to store the IDs of users who clicked or viewed the message in an array. When the number of viewers matched the group size, the tick would change to blue.
    2. Using event listeners and click tracking: Tried to dynamically track clicks on the tick but faced challenges keeping the state consistent.
    3. Intersection Observer: Experimented with detecting when the tick was in view to automatically trigger a state change.

Challenges Faced

  • I encountered significant difficulties getting the feature to work, despite multiple approaches.
  • The feature required deeper interaction with the chat backend than initially expected, and my attempts to track user views and clicks didn’t yield the desired results.
  • Unfortunately, my team wasn’t able to provide much support, and I had to troubleshoot the issue independently.
  • Although the UI element (the tick) was successfully added, the functionality to turn it blue when viewed by all users was not achieved.

Conclusion

I tried my best to complete the feature, but due to the complexity of tracking views across a group chat and limited support from my team, the feature remains incomplete. I believe that further exploration and collaboration would be required to make it fully functional.


Link to Issue

Closes #20

@khiyami khiyami closed this Oct 20, 2024
@DachiCharkviani DachiCharkviani deleted the rreceipts branch October 20, 2024 13:06
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.

User Story 5 - Implement backend functionality to track when messages are read by users
1 participant