Skip to content
This repository was archived by the owner on Dec 6, 2025. It is now read-only.

Conversation

@Jake-Pullen
Copy link
Owner

Pull Request Type: Refactor

1. Title: Removing most of the logic from the database class

2. Description:

  • Summary: By moving the logic of the various key parts of the application to their own respective class we will be able to expand quicker and reduce tech debt moving forwards.
  • Motivation: Putting all of our core logic into the database class was starting to show cracks, in terms of having a monolithic file instead of organised logic separated files.
  • Proposed Solution: By creating classes for each core part of the system we will be able to have a clearer and smoother path to verity development
  • Testing: ensured all unit tests pass, added files ready for user and category tests, extensive testing (manually) on firefox.
  • Known Limitations/Future Considerations: At this moment, we are still limited only to user and category which is far from done but hopefully this lays the foundation for the rest of the application.

3. Checklist:

[x] I have followed the project's coding style and conventions.
[x] I have written unit tests to cover my changes.
[x] My code has been tested locally.
[x] I have included screenshots/GIFs if applicable.
[x] I have updated the documentation (if necessary).
[x] I have labeled this PR appropriately (e.g., "bug," "feature," "refactor").

Alex Dimmock and others added 29 commits May 28, 2025 16:09
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Jake-Pullen Jake-Pullen added the enhancement New feature or request label Jun 28, 2025
@Jake-Pullen Jake-Pullen merged commit 2e278a1 into stage Jul 8, 2025
1 check passed
@Jake-Pullen Jake-Pullen deleted the scale_up_refactor branch July 8, 2025 06:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants