Updated UI, Integrated api#59
Open
arvind-rana wants to merge 6 commits into
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Gautam7352
requested changes
Jun 11, 2026
Gautam7352
left a comment
Gautam7352
left a comment
Member
There was a problem hiding this comment.
- check comments
- give me exact steps to follow to verify if things work or not
Collaborator
Author
There was a problem hiding this comment.
To verify locally:
Web:
- cd apps/web
- npm install
- npm test
- npm run dev
- Open http://localhost:3000 and confirm the web app loads without errors.
Server:
- cd apps/server
- go mod tidy
- go test ./...
- docker compose up -d postgres
- make migrate-up
- go run cmd/main.go
- Open http://localhost:8080/swagger/index.html and confirm the API docs load.
I also removed the generated Jest cache files from the PR and added .jest-cache to apps/web/.gitignore.
Collaborator
Author
There was a problem hiding this comment.
These files were generated automatically by Jest when tests were run locally. They are cache files used to speed up future test runs, such as transformed TS/TSX files, source maps, haste-map cache, and perf cache.
They are not source code and are not needed in the repository because Jest can regenerate them anytime. I removed them from the PR and added .jest-cache to .gitignore so they won’t be committed again.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Frontend Changes
Landing Page
Performance Optimizations
Implemented virtualization on:
This significantly improves rendering performance and overall responsiveness when handling a large number of users or records.
Lazy Loading Enhancements
Added lazy loading for:
This reduces the initial JavaScript bundle size and improves login page load performance by loading components only after role selection.
Backend Changes
CORS & Middleware Updates
Updated
main.gowith the following improvements:CorsConfigconfiguration for proper resource sharing handling.CorsMiddlewarefunction for centralized and consistent CORS management across the application.