-
Notifications
You must be signed in to change notification settings - Fork 20
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
[ENHANCEMENT]: Refactor project code #26
Comments
@singodiyashubham87 Can you please give some more context or explain a bit more about the issue what are the issues in the current code? |
@JAYBORICHA07 I have assigned you an issue already, Have patience and once your PR is merged for that issue then I will explain and assign this issue to you. |
OKAY SIR THANK YOU |
Can you explain about the refactor more? |
@LakshyaSatpal Sure, Refactoring means restructuring and optimizing existing code without changing its external behaviour.
|
can i work on this issue? |
* contribution guidelines added (singodiyashubham87#22) * contribution guidelines added (singodiyashubham87#23) * Ujjwal/contributing guide (singodiyashubham87#26) * contribution guidelines added * dev.to blog link added
@Sahil-Choudhary-17 Go Ahead! |
The code is properly distributed in pages and components folder. Is there any specific part of the code base that you want me to refactor? |
@Sahil-Choudhary-17 See if you can optimize any part of the application, even small optimizations lead to big changes. |
@Sahil-Choudhary-17 Are you still working on the issue? |
No |
can i work on this? also do you have plan of expanding this app further, creating new modules etc? |
@jayant-vashisth I have no such plan for expanding. |
Refactor the project code files without changing the functionality.
The text was updated successfully, but these errors were encountered: