Skip to content

corrected the folder structure#153

Open
mohdabdullahmeraj wants to merge 5 commits intoTanmayKalra09:mainfrom
mohdabdullahmeraj:folder
Open

corrected the folder structure#153
mohdabdullahmeraj wants to merge 5 commits intoTanmayKalra09:mainfrom
mohdabdullahmeraj:folder

Conversation

@mohdabdullahmeraj
Copy link
Contributor

🚀 Pull Request

📋 Description

This PR refactors the src folder structure to improve organization, scalability, and maintainability. All components have been moved into a more logical hierarchy with separate directories for pages, reusable components, and context providers.

  • Top-level page components are now in src/pages/.
  • Reusable UI components (modals, cards, etc.) are in src/components/.
  • Global state management logic is now in src/context/.

All import paths throughout the application have been manually updated to reflect the new file locations. Additionally, the README.md has been updated with a new "Folder Structure" section to document the changes for future contributors.

Fixes #<145>

🔍 Type of Change

  • Bug fix 🐞
  • New feature ✨
  • Documentation update 📝
  • Refactoring or code improvement ♻️
  • Other (please describe):

🙋 Your Details

🧪 How Has This Been Tested?

I have tested these structural changes by running the application locally and verifying that all functionality works as it did before the refactor. My testing process included:

  • Moving all files to their new respective directories (pages, components, context).
  • Manually updating all import statements across the entire project to point to the new file paths.
  • Running the development server (npm run dev) and ensuring the application compiles without any module resolution errors.
  • Navigating to every page (Home, Dashboard, Goals, About, Contact) to confirm that all components render correctly.
  • Testing all core features (adding/deleting transactions, adding/deleting goals, contributing to goals) to ensure no functionality was broken during the refactor.

✅ Checklist

  • I have read the contributing guidelines.
  • I have followed the code style and linting rules.
  • I have added tests or explained why not.
  • I have updated documentation (if needed).
  • My changes do not introduce any known security issues or vulnerabilities.

@vercel
Copy link

vercel bot commented Jul 28, 2025

@mohdabdullahmeraj is attempting to deploy a commit to the Tanmay Kalra's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jul 28, 2025

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

Name Status Preview Comments Updated (UTC)
smart-log ❌ Failed (Inspect) Aug 11, 2025 1:19pm

@TanmayKalra09
Copy link
Owner

There are lot of errors in Code, Please fix them.
Screenshot 2025-07-28 at 3 16 09 PM
Screenshot 2025-07-28 at 3 16 18 PM

@mohdabdullahmeraj
Copy link
Contributor Author

@TanmayKalra09 I have resolved those, now check

@TanmayKalra09
Copy link
Owner

Please Resolve the conflicts :)

@Tanziha-Sheikh
Copy link
Collaborator

Hello @mohdabdullahmeraj

We have merged some PR sand because of it some conflicts arises. Please solve it.

CC: @TanmayKalra09

@mohdabdullahmeraj
Copy link
Contributor Author

@TanmayKalra09 @Tanziha-Sheikh fixed the conflicts

@TanmayKalra09
Copy link
Owner

@mohdabdullahmeraj Please resolve the conflicts

@TanmayKalra09
Copy link
Owner

Hi @mohdabdullahmeraj, are you working on resolving the conflicts? If we don't hear back soon, we may have to close this PR due to inactivity. Please let us know your status. Thank you!

@mohdabdullahmeraj
Copy link
Contributor Author

Hi @mohdabdullahmeraj, are you working on resolving the conflicts? If we don't hear back soon, we may have to close this PR due to inactivity. Please let us know your status. Thank you!

yes working on it

@mohdabdullahmeraj
Copy link
Contributor Author

@TanmayKalra09 please check

@TanmayKalra09
Copy link
Owner

@mohdabdullahmeraj The Vercel deployment has failed and points to some errors. Please resolve them, or contact @Tanziha-Sheikh — she will guide you.

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.

3 participants