Skip to content

Reduce/fix backend duplication #346

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

Draft
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

carolinaisslaying
Copy link
Member

@carolinaisslaying carolinaisslaying commented Apr 23, 2025

DRAFT

Description

This aims to reduce all (or at least significantly reduce) code duplication, as well as splitting up files things into separate files where appropriate (i.e., better split up staff endpoints, etc).

Type of Change

  • Feature enhancement
  • Bug fixes
  • Refactor
  • Breaking change
  • Performance improvement/reducing time complexity

Additional Information

This PR is a draft, thus work is still ongoing. Until it is un-marked as a draft, things in this branch/PR may change at any time, and this description may also change at any time.

Confirmation Checklist

  • I have ensured that my code follows the project's style guidelines (pnpm run pretty).
  • I have performed a self-review of my code and corrected any issues.
  • I have tested the changes to verify they work as intended.
  • I have confirmed that my changes do not introduce new security vulnerabilities.
  • I have documented and communicated any breaking changes appropriately.

@carolinaisslaying carolinaisslaying added this to the 5.7.0 milestone Apr 23, 2025
@carolinaisslaying carolinaisslaying added the refactor Issue/PR related to refactoring. label Apr 23, 2025
IceeMC added 5 commits May 16, 2025 13:37
- Properly revoke admin tokens for previous admins.
- Fix error handler.
(cherry picked from commit 2a0c84c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Issue/PR related to refactoring.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants