issue76 - Server improvement + problems solved

JaredHawkins / TweetGeoViz

Visualization tool to view tweets by location and content.

Apache License 2.0

25 stars 16 forks source link

issue76 - Server improvement + problems solved #79

Closed carloscarcamo closed 8 years ago

carloscarcamo commented 8 years ago

Requests without searchQuery returning an empty array of results.
MongoDB connection issues, now reconnects to mongo after connection loss.
Organizing databases files in to 'store' folder.
Moving tweets collection query methods into a separate file.

anvk commented 8 years ago

This pull request looks great !!! Thanks for making those changes

carloscarcamo commented 8 years ago

@anvk changes done! check if it is ok now :)

anvk commented 8 years ago

Hey @carloscarcamo great job! sorry to bug but there is one more little change and I will merge it in.

carloscarcamo commented 8 years ago

@anvk no problem! check now please!

Can you create a code conduct or code standard for this project? Would be good to have such thing for everyone who wants to contribute to this project.

anvk commented 8 years ago

Merged! Well not sure if I need to add this code conduct page. There are so many available online already. For example https://github.com/airbnb/javascript This is a very popular source especially since it is used by many many projects while eslint-ing. Maybe in the future I just add eslint to the project which would lint the code for any "bad syntax" and errors.