Closed ayushseth07 closed 3 years ago
@ayushseth07 I noticed that there is already a showPassword.js
file. So you can put your new function in this instead of creating a showPassoword2.js
.
I asked you to create a new .js
file last time. Sorry for the wrong suggestion. I didn't know that showPassword.js
already existed.
Also, you don't need to create a new PR every time. Just update the local branch and git push
which will automatically update this PR.
@ayushseth07 Everything looks amazing!! Did you test the working of new changes?
@siv2r yes i have tested them and they are working fine
Congrats 🎉 on merging your first pull request. Congratulations on merging your first contribution to kidney-exchange✨
@ayushseth07 Thanks for this amazing PR :tada: :tada:
(Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.)
fixes #324