-
Notifications
You must be signed in to change notification settings - Fork 0
Merge to main for deliverable 4 #124
Merged
+10,363
−838
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
…igation-and-improve-navbar 86 replace query parameter navigation and improve navbar
…tions-and-formatting 102 standardize naming conventions and formatting
…n the buil, also implemented fetching specifications only when needed (add, partview), and added lazy loading of images
… dialog is shared over all parts instead of one for each part.
…, also fix a bug wher if part details where fetched when showing details they were lost when exiting that dialog
…same filters within a minute will now fetch data from a cache instead of the backend.
…he state of the current prototype
…fic page sets the category to that one
…hat given proper user feedback
…ons-and-usability 101 improve catalog interactions and usability
…dency-vulnerabilities 103 fix backend runtime dependency vulnerabilities
…and updating FilterComponent and NavBar to use it
…ications-are-showing 111 fix no scroll while notifications are showing
…, and a test for it
…pc-components 114 add skeleton loading for pc components
- Implemented a new GraphQL query resolver for fetching builds by ID with validation. - Added unit tests for the build resolver, covering valid and invalid ID scenarios. - Updated package.json to include vitest for testing. - Created a vitest configuration file for test execution. - Adjusted tsconfig.json to include source files and exclude test files.
…dd comprehensive tests
… into 92-add-automated-tests-for-critical-functionality
…add IP context to publishBuild mutation
…data handling and UI updates
…nonymous-build-publishing feat: add abuse protection and request limits for anonymous build publishing
…ompatibility-panel 90 improve text contrast in compatibility panel
…itical-functionality 92 add automated tests for critical functionality
…-query-performance 100 improve community backend query performance
…lity-and-user-feedback 99 improve pc builder reliability and user feedback
- Added OpenSavedBuild component to handle loading saved builds in the PC Builder. - Updated PcBuilderPage to integrate OpenSavedBuild and display editing status for saved builds. - Created AccountPage to manage user accounts and display their builds. - Implemented DeleteBuildButton for deleting builds with confirmation dialog. - Enhanced backend tests for build ownership and authorization. - Expanded frontend tests for saved build workflows and delete functionality. - Updated README with new test results and coverage.
…to 97-implement-user-authentications-and-build-ownership
…ions-and-build-ownership 97 implement user authentications and build ownership
…ocumentation docs: update README and add accessibility and sustainability document…
oscarog
approved these changes
Nov 13, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Sign in
to join this conversation on GitHub.
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.
Purpose
The PR is necessary to get the up to date changes to the main branch for review of deliverable 4.