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

Added paginate function to help users paginate through database results. #161

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

davidw
Copy link
Contributor

@davidw davidw commented Jan 27, 2014

Sorry, but I was in git hell, and decided to destroy my old boss_db fork and start anew to make things clean.

@davidw
Copy link
Contributor Author

davidw commented Jan 27, 2014

Regarding testing, I'm not really sure what is going on in boss_news_controller_test.erl, so... yes, I need some help to add the tests.

@zkessin
Copy link
Contributor

zkessin commented Jan 27, 2014

Ok, call me on skype and we will work on it, can you do tomorrow?

@zkessin zkessin closed this Jan 27, 2014
@zkessin zkessin reopened this Jan 27, 2014
we now escape column names in inserts and updates and escaping is done right on input strings

fix ErlyORM#164
@davidw
Copy link
Contributor Author

davidw commented Jan 27, 2014

Easiest thing for me is to coordinate on #ChicagoBoss - I'm not 100% what my schedule is like tomorrow.

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

Successfully merging this pull request may close these issues.

2 participants