-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: fixes and validation #35
Merged
Merged
Conversation
This file contains 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
* fix: infinite looping cronjob * feat: rewrite service unit tests * feat: add global err handlers * feat: add 404 * feat: update api client * refactor: use common page skeleton on loading * fix: fix code smells * feat: add ci badge * feat: add ratelimit handling * fix: fix proxy security warn * fix: typo * refactor: auto detect rsc on xhr request * feat: rewrite auth tests * chore: bump deps, fix text * feat: start on announcementmodel unit tests * fix: add cleanup func
* feat: write middleware for checking * feat: overhaul file structure * feat: upgrade auth endpoints * feat: add typecheck for /user/* * fix: provide full list of missing env * feat: announcement api validation * feat: upgrade service endpoints * feat: fix up announcementmodel tests
* feat: migrate db * feat: update models and endpoints * fix: update tests * feat: update scheduler * refactor: deletion logic efficiency * fix: debounce state for xhr req * feat: update FE service sessions and varifyattendance * feat: add support for service * feat: improved validation errors * chore: rename service hours to cca hours * fix: fix smells
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.
This PR implements frontend cleanup and api validation.