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

🧼 Cleaner architecture #91

Merged
merged 34 commits into from
Aug 31, 2024
Merged

🧼 Cleaner architecture #91

merged 34 commits into from
Aug 31, 2024

Conversation

runeharlyk
Copy link
Owner

Purpose

Implements a cleaner architecture

The actions to obtain this goal includes, but not limited to:

  • Collects all api definition in webserver.cpp, making it simpler to see what implement, writing docs and general overview.
  • Organizing different components to folder, making it easier to find files
  • Refactors a lot of service be one stateful service, with setting and functions.

This should:

  • Make it easier to test and maintain
  • Make it easier to make spot a framework
  • Make it easier to add new service

What needs to be done

[ ] - Collect all socket handlers and event definition
[ ] - Collect all file locations

@runeharlyk runeharlyk merged commit 7f6202b into master Aug 31, 2024
2 checks passed
@runeharlyk runeharlyk deleted the cleaner-architecture branch August 31, 2024 21:59
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.

1 participant