Skip to content

Conversation

@ThePhoenix08
Copy link
Contributor

Description

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
  • regenerated package-lock.json and package.json for both directories

Related Issues

Relavant 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.

Note to reviewers

I am contributor under SWOC. Please review my PR and assign me appropriate points on merge.

(cherry picked from commit 4cf953a)
(cherry picked from commit ff305e1)
(cherry picked from commit ab8156e)
(cherry picked from commit 6c6a201)
(cherry picked from commit d50b032)
(cherry picked from commit 071f420)
(cherry picked from commit f730f3d)
(cherry picked from commit 3a15f09)
@bolt-new-by-stackblitz
Copy link

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

@vercel
Copy link

vercel bot commented Jan 16, 2025

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

Name Status Preview Comments Updated (UTC)
devdisplay ❌ Failed (Inspect) Jan 16, 2025 10:02am

@codeaashu codeaashu self-requested a review January 16, 2025 10:30
@codeaashu codeaashu added DevDisplay 🌟 This label recognizes every contributor to the DevDisplay Community for their support...💌 Dev Enhancer 🌟 Label 2 for second-time contributors who enhancing and refining features of DevDisplay. 🔧 Dev Innovator 🌟 Label 3 for contributors adding new, innovative features to DevDisplay, leading in creativity.💡 SWOC Label is used for accept the pull request of contributer in SWOC 2025 swoc25 Label is used for accept the pull request of contributer in SWOC 2025 labels Jan 16, 2025
@codeaashu codeaashu merged commit 4930e1a into codeaashu:main Jan 16, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dev Enhancer 🌟 Label 2 for second-time contributors who enhancing and refining features of DevDisplay. 🔧 Dev Innovator 🌟 Label 3 for contributors adding new, innovative features to DevDisplay, leading in creativity.💡 DevDisplay 🌟 This label recognizes every contributor to the DevDisplay Community for their support...💌 SWOC Label is used for accept the pull request of contributer in SWOC 2025 swoc25 Label is used for accept the pull request of contributer in SWOC 2025

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants