Skip to content

feat: create api and websocket managers #199

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

Merged

Conversation

egenerse
Copy link
Collaborator

@egenerse egenerse commented Jun 9, 2025

Checklist

  • I linked PR with a related issue

Motivation and Context

This PR completes #198
This PR completes #200

We want to manage websockets and API calls in their own managers to separate concerns. This makes ApollonWithConnection Component lighter and easier to read

Also adds a scheduled cron job that runs every day at midnight to automatically delete diagrams from the database that haven't been updated in the last 60 days. This helps keep the database clean by removing stale or unused data.

Description

DiagramAPIManager is created to handle API calls in one centralized place
WebSocketManager is created to keep Websocket logics
Also created a cron job to delete diagrams untouched last 60 days.

Steps for Testing

  1. Create a diagram
  2. Share it in collaboration mode
  3. Collaborate on same diagram
  4. Observer share and collaboration works normal

Screenshots

@egenerse egenerse requested a review from kurunbelemir June 9, 2025 11:48
@egenerse egenerse self-assigned this Jun 9, 2025
@egenerse egenerse added the build and push docker image Build and pushe docker image with tag label Jun 9, 2025
@egenerse egenerse merged commit 78b9a75 into main Jun 9, 2025
3 checks passed
@egenerse egenerse deleted the feature/create-webapp-side-API-and-Websocket-managers branch June 9, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build and push docker image Build and pushe docker image with tag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants