-
Notifications
You must be signed in to change notification settings - Fork 32
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 / rewrite Upload_ReactComponent.react.js #22
Comments
I would like to help, but I have no idea where to start. Is there a description on how to setup the development environment? |
I'm very happy to hear! You can read the instructions for setting the development environment in the CONTRIBUTING.md. Please just let me know if you need any other guidance. |
Is this still being worked on? If not I would like to pick it up. |
I have not done anything for this but I don't know if @soerendip has. Great to hear @Vipulsh if you want to help! |
Alright, I will set up the dev environment first and then start working on it. |
I have created a pull request after making some changes. |
Great! Thank you @Vipulsh, I will have a look and do some testing once I have some time :) |
There has been some new merges (e.g. #26), and the most up-to-date refactored branch is the v.0.5.0 release candidate: https://github.com/np-8/dash-uploader/tree/0.5.0 Some more refactoring could still be done. It might be useful to refactor with #21. |
The
Upload_ReactComponent.react.js
would really need some refactoring. This would make further development a lot easier and the package more maintainable.The text was updated successfully, but these errors were encountered: