Skip to content

Sprint3 - Notifications and realtime updates (sockets)#23

Merged
BennettTheTiger merged 16 commits intomainfrom
sprint3
Apr 6, 2025
Merged

Sprint3 - Notifications and realtime updates (sockets)#23
BennettTheTiger merged 16 commits intomainfrom
sprint3

Conversation

@BennettTheTiger
Copy link
Copy Markdown
Owner

@BennettTheTiger BennettTheTiger commented Apr 5, 2025

  • add sockets with real time sync
  • new/updated unit tests
  • FP Tree algorithmic element 🧠

@BennettTheTiger BennettTheTiger self-assigned this Apr 5, 2025
Copilot AI review requested due to automatic review settings April 5, 2025 01:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 18 out of 18 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

routes/v1/nameContext/index.js:16

  • The imported middleware 'checkNameContextOwnerOrPartipant' has a typo. Consider renaming it to 'checkNameContextOwnerOrParticipant' for clarity and consistency.
const checkNameContextOwnerOrPartipant = require('../../../middleware/checkNameContextOwnerOrParticipant');

@BennettTheTiger BennettTheTiger merged commit 2e945c7 into main Apr 6, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants