Skip to content
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

Directory Reorganization and Minor Fixes #707

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

ThePhoenix08
Copy link
Contributor

@ThePhoenix08 ThePhoenix08 commented Jan 4, 2025

Description: Directory Reorganization and Minor Fixes

Summary of all changes: -

  • Re organised directory structure into /frontend and /backend
  • fixed minor eslint and other issues with frontend
  • Implemented a basic express backend to begin with, some helpers and /health route
  • added user model

Related Issues

Relevant Issue #692

Changes Proposed

/ frontend changes

  • fixed ambiguity of "nishantrana07.json" and "NishantRana07.json", correct is "NishantRana07.json"
  • restructured the projects folder structure - /frontend and /backend introduced with respective files moved
  • fixed some useEffect dependency issues and implemented useCallback and useMemo wherever required to fix eslint warnings in frontend
  • Updated packages and fixed fixable issues using npm audit fix and npm update in frontend

/ backend changes

  • Initialised backend and added a /health route to start with

  • Wrote helpers for standardized response and error formats

  • added User model

  • pulled recent changes manually to /frontend

  • regenerated the package.lock json for both frontend and backend

Checklist

  • I have read and followed the Contribution Guidelines.
  • All new and existing tests passed.
  • I have updated the documentation to reflect the changes I've made.
  • My code follows the code style of this project.
  • The title of my pull request is a short description of the requested changes.

Screenshots

NA

Note to reviewers

This is a breaking change.
It will affect production aswell.

Copy link

stackblitz bot commented Jan 4, 2025

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

Copy link

vercel bot commented Jan 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
devdisplay ❌ Failed (Inspect) Jan 4, 2025 1:37pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant