-
Notifications
You must be signed in to change notification settings - Fork 6
Server rewrite #1030
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
lhvy
wants to merge
33
commits into
dev
Choose a base branch
from
server-rewrite
base: dev
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
Server rewrite #1030
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
Contributor
Author
|
CI is failing without code changes. Not good. |
* Add routes for adding and removing course * Update routes by moving validation to controller * Add route for update course colour * add OpenAPI for api-docs for backend using swagger library * Add ApiProperty for user's route * Add get route for courses and classes with update of routing urls * Update PUT request for setCourseColour to PATCH request * Update user routes to using AuthenticatedRequest Type * Add selectClass & removeClass route * Remove OpenAPI and Swagger Library, switched to use Postman * Update route for select and remove class * Remove unwanted routes with forwarding functions * Resolve usage of null and undefined * Resolve PR review comments * Slim down the Course and Class detail for graphql, and add validation function for update and remove class * Update routes returning status code * Refactor user.controller, move all business validation into service level * Remove duplicate getCourse request in addSelectedClass * Resolve issues on the PR code review * Resolve minor issue on function call, optimise prisma call * Resolve function name typo * Update service level function name corresponds to control level
* Remove old account implementation * Delete group sidebar * Remove timetable syncing and groups
* Add authentication guard and login modal * Fix login redirect * Fix casing on pathname of png
* created routes for user timetables * fixed returns * separarted timetable and course routes into the new structure * removed redundant imports * fix redundant assert --------- Co-authored-by: sunny chen <[email protected]>
* Install TankStack library with corresponding devtool, integrate into App.tsx * Integrate react query in to setting page * Connect settings.useSquareEdge,useDarkMode,preferredTheme with FE components * Remove use of QueryContext to direct suspense query calling to avoid re-rendering of whole App component * Update all settings to use React Query * Fix FE build error by removing removed AppContext states on components * Remove redudant comments * Add Error Boundary for handling unintented error, resolve packages version and dependecies issue * Remove unwanted comments and resolve naming issues * Resolve setting item destructing before used, and other dependecies issues * Migrate JS fetch to Axios * Remove unwanted library import * Update ErrorBoundary to handle any unknown errors * Resolve build error * Rebase from server-rewrite * Rebase from server-rewrite, with autofix eslint * Refactor Settings component to fit the feat of profile picture * Remove unneccssary commnets * Resolved dropcard shadow logic with react query * Resolve CI build error and lint warnings from drag.ts * Resolve review errors from lucas * Resolve use-prefix issue for settings parameter with minor changes * Resolve review comments from mark * Update the prisma schema for user settings * Update the prisma schema for user settings * Resolve naming error
* Upgrade to MUI X 7 * Upgrade to MUI 6 * Upgrade to MUI 7 * Upgrade to MUI X 8 * Bump bug dependencies * Bump minor dependencies * Upgrade Sentry * Upgrade plugin-react-swc and web-vitals * Bump Apollo to 3.14 * Apply pnpm update * Fix Apollo client version * Upgrade to date-fns 4 * Bump bug and minor dependencies * Fix UI bug from Grid after MUI 7 migration * Fix mui style of using sx prop instead of flexGrow --------- Co-authored-by: jason4193 <[email protected]>
* Refactor auth to support multiple sign in methods * Add guest user support * Add todo comments for future guest user changes * Add GitHub to the auth provider enum
* upgrade apollo client to 4.0.4 with codemod migration * Fixed the FE build error by proper typing for the graphQL queries * Remove redundant comments
* Migrate to React 19 * Fix null issue with drag and drop * Clean up rippleRef usages * Fix ripple effect on the first click * Bump minor and bug frontend dependencies * Upgrade to React Router 7 * Bump minor and bug frontend dependencies
* Add GitHub auth * Add Google auth * Enable frontend auths button * Fix bug with Google and GitHub passport login * Refactor and add logout route * Add logout button to frontend * Fix logout redirect * Fix hero section button when logged in * Fix props after rename * Apply lint fixes
* Update back the routes of course and class into timetable.controller * Add timetable init for oidc user if no term timetable * Resolved PR review from lucas (implement onboard util function) * Update the logic for create timetable using graphql query * Update google and github auth strategy with userOboard function * Resolve review from lucas
* Remove old friends implementation * Add togglable friends * Move sidebar collapsed state to app context * Remove another old friend component * Add friend list and search bar * Refactor sidebar's collapsed state into an AppContext * MVP of friends list with simple search * reverting auth guard lol * Refactor sidebar styled and fix user account padding * Fix friends list scroll issue * Fix sidebar always scrolling * Remaster the add friends button * Move scrollable component down by 1 layer so that notangles logo is not part of scroll * Add todo comment * undo changes for auth on server * Fix search and add friend buttons on sidebar for collapsed. Also added hover to support future click-on-friend operation * Clicking search button on collapsed sidebar will expand it and focus on form immediately * Hide sidebar scroll (removes weird ui bug in collapsed as well) * Remove empty tag wraps --------- Co-authored-by: Mark Tran <[email protected]>
Co-authored-by: Mark Tran <[email protected]>
* Refactor custom modal to have modal reusable * Add friends modal menu * Add invite code, send request and copy link section of menu * Implement hard-coded copies and alerts * Make borders thinner * Fix snackbar to be closer to add friend menu * Clean up css * Revert back to prev Paper component since it's not happy with div elements for form submissions * Fix variable naming for SettingText --------- Co-authored-by: Mark Tran <[email protected]>
* Fix the timetables routes to handle by Id & Set init timetable to primary * Use term _eq instead of _like for addCourse * Remove unwanted error log
* Refactor settings panel to be flat menu * Add feature to change and remove profile photos (for logged in users) * Fix forward icon and remove duplicate arrow icons in photo upload modal * Add logic to hide profile pic menu option if user is guest * Clean up profile pic components --------- Co-authored-by: Mark Tran <[email protected]>
* Add orutes in controller for events * Resolved conflict * remove need to timetable id * Add get all events * Resolved conflict * Resolved conflict * Fix remove event function * Resolved conflict * Fix delete route and fix ids for events * Remove excess file * Resolved conflict * Fix linting * Basic changes made from review comments * Fix events info except for type parameters * Fix most comments * Change service to use prisma generated data * Fix linting * Fix minor issues with routes * Fix lint * remove console log * Fix env ecample * Fix lint * Fix lint * Add user validation to update event * Clean up Prima schema and migrations
* Added username shortening functionality * Update client/src/components/sidebar/friends/UserProfile.tsx Fix formatting on getTextWidth function Co-authored-by: Mark Tran <[email protected]> * Remove canvas element from DOM after its used to check text width --------- Co-authored-by: Meredith Zhang <[email protected]> Co-authored-by: Mark Tran <[email protected]>
* Basic start code for clear timetable * Passing linting * Fix clear timetables * Fix linting * Fix userId * return new id and change names * Change variable to underscore
* make server listen on 0.0.0.0 to accept connections from any external devices * Added pending friend invites badge with hardcoded invite values. Changed friends icon to a group icon instead. * Reverted back to localhost binding * Swapped to use red theme colour instead * Added pr review fix
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.
TBD