-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update the dependency #1
Comments
May I take this issue? ^^ |
I am going to work on this if that's okay? |
@NeonNature Sure go ahead. This code is very old. So do whatever you feel like. |
Oh man I forgot to reply to this issue. I am already working on it |
I'm wondering why |
I did npm audit fix, updated Express and Mongoose to current version. I also fixed the database connection using some of the settings recommended by Mongoose in order to hide warning messages about deprecation. Also, I updated the bootstrap and the layout of the page. Not so much difference though, but at least it has some responsiveness now. Maybe the PR from @NeonNature will conflict my PR if you approved his PR, so let me know when I should fix the merge conflicts. Thanks |
@dennypradipta i have merged @NeonNature PR. So please go ahead and check if is there any conflict. |
Ok I forked again, and made some changes. Take a look |
Thank you for your contribution guys |
Hi there first timers,
Here is a simple task for you.
I haven't updated the dependency and code for a while and it can be improve. So Pull this repo and update and fix what ever you feel like fixing. and push to merge.
All the best,
Shivam Gupta
The text was updated successfully, but these errors were encountered: