vinitshahdeo / PortScanner

A go-to tool for scanning network. Scan all the open ports for a given host with just one click.
https://vinitshahdeo.github.io/PortScanner/
MIT License
125 stars 114 forks source link

Port Scanner Form Component in React #121

Open ishika1727 opened 3 years ago

ishika1727 commented 3 years ago

As we've built many features to justify the one-liner description of this project i.e. a go-to networking tool, now it's time to revamp the existing UI.

Add a new component PortScannerForm.js which will take inputs for:

This component should also have a button called SCAN. The scanning for open ports will start on clicking this button. Refer the image attached below for design

As of now, on clicking SCAN button, the values in the input fields should be displayed in console

P.S. : The basic frontend on react is available on feature/react branch in the folder network-scanner. Add your components in the same branch. Do not make any changes in App.js / index.js.

image

devRawnie commented 3 years ago

I would like to work on this. Please assign this to me.

subhangi2731 commented 3 years ago

can you please assign it to me?

Abhishek765 commented 3 years ago

I would like to work on this Please can you assign me? I guess your branch feature/frontend-react is not merged currently so can I create a new react app network-scanner and add the PortScannerForm.js component in that?

vinitshahdeo commented 3 years ago

Hello guys, The branch will be merged soon. Review is still in-progress. Post that we'll assign the issues.

vinitshahdeo commented 3 years ago

@ishika1727 I have merged the PR. You can assign this issue with a story point of not more than 3 days.

ishika1727 commented 3 years ago

@devRawnie Please go ahead with the issue and create a PR in feature/react branch by Monday EOD.

Feel free to contact me anytime

devRawnie commented 3 years ago

Sure thing. I am on it

devRawnie commented 3 years ago

Can I use reactstrap library to style the input fields?

ishika1727 commented 3 years ago

@devRawnie yes, please go ahead, but try to keep it as simple as possible as we will be merging all components together after they have been made.

devRawnie commented 3 years ago

How do I add reviewers in the pull request, I couldn't figure it out while creating the pull request.

ishika1727 commented 3 years ago

I ll add the reviewers, no issues

ishika1727 commented 3 years ago

@devRawnie Merged your PR into feature/react branch. Amazing work. Please go through the other open issues as well and contribute more to this project