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

improve structure of application #33

Open
Curiouspaul1 opened this issue Oct 11, 2021 · 2 comments
Open

improve structure of application #33

Curiouspaul1 opened this issue Oct 11, 2021 · 2 comments

Comments

@Curiouspaul1
Copy link
Contributor

Hi!, really nice work you have here. I suggest you work to improve the structure of the application, you can try using classes and helper methods to get things more organized, that'll reduce the number of files you need, and keep everything organized and easy to understand.

Also, it seems like the requirements.txt file has some unneeded dependencies in it, whoever made it must have unknowingly added them to it. For example the requirements file specifies Django as a dependency and i don't see any reference to django in your code base. You should look into that as well.

@Curiouspaul1
Copy link
Contributor Author

I would be willing to work on all these issues mentioned if you can assign it to me

@GauravSingh9356
Copy link
Owner

@Curiouspaul1 go ahead dude

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

No branches or pull requests

2 participants