-
Notifications
You must be signed in to change notification settings - Fork 62
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 dependencies #9
base: backend-starter
Are you sure you want to change the base?
Update dependencies #9
Conversation
Version 7 of pg wouldn't connect to latest postgres databases
Incidentally, I started the last week-end myself to update the old GitHub version to current Node 12 LTS and aiming to use PostgreSQL 12. The idea is to pull the versions to the state of the art versions for newcomers. Some general comments on potential improvements/fixes: The blog example would be far easier for newcomers to install if we would provide something basic Linux (e.g. Ubuntu) examples: And last but not least, we should exchange 'patrick' in the SQL dump and the examples with 'gameadmin', so no one gets confused and everyone might keep the user. Perhaps we should generate some docu equivalent to the blog posts in the GitHub Docs dir from markdown or similar. What do you say, James? Or anyone, even Patrick, who seems to be quite busy these days ;-) |
@svanteschubert I certainly think there's lots that could be updated. It's such a useful little tutorial that it would be a shame if people were put off because of things not working. Took me quite a while to get things sorted (the tileLayer url is out of date too for example) that it would be good to help others not make the same mistakes! It all depends on whether @triestpa might have some time to review PRs etc and update the blog post |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey folks, I'm sorry for the late reply here! This looks good to me, I'll make a not to update the tutorial with these changes soon too
If you have any other fixes locally (such as tileLayer url update) I'd be happy to merge them in and to update the tutorial text for it as well. |
Ive been going through this project. Its quite impressive! Im coming across out of date packages also. Will this pull request be merged? |
@triestpa Patrick, it seems you have forgotten to merge this one. :-) |
Version 7 of pg wouldn't connect to latest postgres databases. Took me a long time to work out what was going wrong, before realising that the dependencies were quite out of date. These updates should all work fine.