-
Notifications
You must be signed in to change notification settings - Fork 1
feat: API rework #220
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
RappyTV
wants to merge
50
commits into
development
Choose a base branch
from
feat/api-rework
base: development
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
feat: API rework #220
+5,089
−2,759
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add gift code ids * Add creator field * Improve creation and deletion staff log message * Add #getCreatorProfile and #usesLeft methods
This was
linked to
issues
Jun 13, 2025
* Add database schemas * Add API routes
* Pass session player via middleware directly
feat: Rework permissions
* Add id field * Add optional color field
* Accept role object as data parameter instead of only role name
* Return new fields * Allow role editing (partially) * Fix role deletion
* Add missing category edit route * Add some todos * Fix some documentation issues
* Add staff translations * Remove unused translations * Fix some translation keys
feat: Database and route improvements
* Fix typo in translation key * Add missing translation key
* Add warning and sentry error if i18n call is not prefixed with $. * Prefix all translation strings with $.
feat: Add translation validation workflow
* Move reports from players into own schema * Add more functionalities * Implement creation in Player#createReport
* Update report route, also return report id in success message * Add routes to get reports made by a specific player * Improve report log with direct report document reference
* Rename exports * Add docs * Add gift type enum
* Add docs * Rename exports * Add staff member methods
* Improve metrics, codes, reports and staff categories
* Add docs * Rename exports * Rename fields and methods * Add a lot of new features * Remove connections * Deprecate some stuff * Add per-role visibility
* Improve typings * Avoid code duplication * Add descriptions and default values * Add docs to role routes * Add enum for documentation categories * Deprecate old tag, position and icon change routes
* Used for certificate, mail templates and custom icons * Create enum for mail templates
* Don't require fields with nullable values * Reformat some schemas * Add Player#changeTag
* Merge tag, position and icon change routes into PATCH /players/:uuid route * Rename icon route into plural * Create route which creates an empty account
- Add some todos - Remove unused imports
feat: Update database schemas and routes
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
API Coverage