-
Notifications
You must be signed in to change notification settings - Fork 52
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
The routes classification should be improved #87
Comments
I would love to work on this issue, but i'm a beginner, any pointers please? |
@abdulsamad234, good to see your interest in the project. You can consider these resources.
|
Okay sir |
can i work on this. i have worked a lot with nodejs and express and this seems like a easy thing to do. |
@abhanshgiri21, sure have a go at it. But make sure you update the API calls from the views too. |
ya sure, i will test everything out too, making sure anything doesnt break. Will update when i am ready to make a pull request. |
can you do one favpur, there are different routes in admin.js as well as index.js . Can you make a tree kinds of, starting from app.js and then classifying all the routes and mount points. you can use coggle, its a great tool for doing such work. |
Can I work on this? |
I'm submitting a ...
Current behaviour:
The routes are not properly classified
Expected behaviour:
There should be separates routes for these
They should be mounted on their own urls. And the current functions should be moved to their corresponding files.
The text was updated successfully, but these errors were encountered: