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

Adding a validation to the inputs #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adding a validation to the inputs #8

wants to merge 1 commit into from

Conversation

romavini
Copy link

@romavini romavini commented Oct 8, 2020

Hi, I added validation to the entries, to prevent the application from closing due to mistyping.

@Attupatil
Copy link
Owner

Hello @mdcdxcvi I have added this as Hacktoberfest-accepted. This is same as accepted the contribution and you will see this in your hacktoberfest profile soo don't worry.
will check with the code soon.
Thank you for contributing for open-source.

@Attupatil
Copy link
Owner

@mdcdxcvi, hope you have received pull request reward on hacktoberfest profile by keeping it as hacktoberfest-accepted
please reply.

@romavini
Copy link
Author

Hello @Attupatil

Yes, I received the contribution at Hacktoberfest, thanks! I hope the changes will help the project.

Check, when you can, also PR #7

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

Successfully merging this pull request may close these issues.

2 participants