-
Notifications
You must be signed in to change notification settings - Fork 94
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
Refactor large components #108
Comments
Hey @sjha2048 , I would like to know more on the issue . |
Hi @sjha2048 @Shruti3004, If no one's working on this issue then can I take this up? There are many open issues on this repo and many people have already commented for taking up the issues. But as of now, those requests are pretty old and still, no one has been assigned to the issue. So please let me know if the project is still accepting contributors. |
hi @Raunakk02, indeed if there is no working on the issue, your contribution is welcome. Please keep in mind that refactoring can be difficult to describe, maybe that way a reason for @sjha2048 to hesitate answering. |
Thanks for the response @mcjaeger. Really appreciate it! |
Hi @Raunakk02, sorry somehow I missed your reply. feel free to take up this issue and reach out to me in case of any issues/discussions, for now, I'll assign this to you. |
Hi, in case you would have the time, maybe consider outlining what your plans are? So we could give feedback and adjust in case. By that approach you re avoiding to work on a PR for a long time alone and then publish something which overlooked one thing in the beginning. |
Sure @mcjaeger, I'll let you know about my plans once I am done outlining them. |
@mcjaeger @sjha2048 Here is the outline of the work that I am planning to do for the PR targeting this issue. Refactoring outline
Please let me know your views on this. |
Hi @Raunakk02 Thank you for the pull request, I'll drop my review soon. |
Description
There are several components which needs to be refactored/broken into smaller components
The text was updated successfully, but these errors were encountered: