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

Add proper UserController #59

Open
JainPriya1234 opened this issue May 31, 2023 · 5 comments
Open

Add proper UserController #59

JainPriya1234 opened this issue May 31, 2023 · 5 comments
Assignees

Comments

@JainPriya1234
Copy link

Describe the bug/feature

controller for user is written in models folder .So,I want to create separate controller folder to handle user data

To Reproduce

  • Steps to reproduce the behavior:

Expected behavior

  • A clear and concise description of what you expected to happen

Logs

  • if applicable, add logs to help explain your problem (e.g. the error message and/or exception traceback).

Environment (please complete the following information):

  • OS:
  • Browser:

Additional Context

  • Add any other context about the problem here.
@JainPriya1234
Copy link
Author

@jain-rishabh-21 @Gurdeep475 please assign this issue to me

@apeksha235
Copy link
Collaborator

apeksha235 commented Jun 1, 2023

hey @JainPriya1234 , could you also add the steps to reproduce it, so I can know how you will be proceeding ahead with it. thanks.

@JainPriya1234
Copy link
Author

hey @apeksha235 ,

  • I want to make a proper MVC folder structure that is followed in node.js.
  • As it will help in debugging and modification quickly.
  • And help others to contribute easily.
  • As they can add features to the project easily because of the proper MVC(Model View Controller) folder structure.

imagee123

@JainPriya1234
Copy link
Author

@apeksha235 please review my approach

@apeksha235
Copy link
Collaborator

@JainPriya1234 , I have assigned the issue to you, you can go ahead

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 a pull request may close this issue.

2 participants