Skip to content

[OA][Fullstack] - Welcome to Sarge, Websockets ( Node )#260

Merged
LOTaher merged 2 commits into
mainfrom
ws-node
Mar 24, 2026
Merged

[OA][Fullstack] - Welcome to Sarge, Websockets ( Node )#260
LOTaher merged 2 commits into
mainfrom
ws-node

Conversation

@cherman23
Copy link
Copy Markdown
Collaborator

@cherman23 cherman23 commented Mar 23, 2026

[OA][Fullstack] - Welcome to Sarge, Websockets

Changes

Scaffolded our new websocket server to handle OA clients.

  • Written in Node
  • Containerized and implemented into our docker-compose.yml file.
  • ./start.sh script now runs the websocket server.

Notes

Currently, the websocket server simply echoes the message sent from the client.

Websockets can be tested locally via Postman.

  • Press New
  • The server listens on ws://localhost:8080
  • Press Connect
  • Type any message and press Send

Screenshots

image ---

Checklist

Please go through all items before requesting reviewers:

  • All commits are tagged with the ticket number
  • No linting errors / newline warnings
  • All code follows repository-configured formatting
  • No merge conflicts
  • All checks passing
  • Screenshots included for UI changes
  • Remove non-applicable sections of this template
  • PR assigned to yourself
  • Reviewers requested & Slack ping sent
  • PR linked to the issue (fill in 'Closes #')
  • If design-related, notify the designer in Slack

Closes

Closes #250

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sarge Ready Ready Preview, Comment Mar 24, 2026 0:50am

Request Review

@cherman23 cherman23 changed the title feat: WebSockets in Node [OA][Fullstack] - Welcome to Sarge, Websockets ( Node ) Mar 24, 2026
@cherman23 cherman23 marked this pull request as ready for review March 24, 2026 00:51
@cherman23 cherman23 requested a review from LOTaher March 24, 2026 00:51
Copy link
Copy Markdown
Collaborator

@LOTaher LOTaher left a comment

Choose a reason for hiding this comment

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

nice! thanks carter

@LOTaher LOTaher merged commit 4154aaa into main Mar 24, 2026
4 checks passed
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.

[OA][Fullstack] - Welcome to Sarge, Websockets

2 participants