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

Suggested change to include a limit to the number of comments returned, ... #2

Closed
wants to merge 1 commit into from

Conversation

tselaty
Copy link

@tselaty tselaty commented Jul 6, 2012

...which serves purpose in the case of a "LOAD MORE COMMENTS" button with Ajax.

…d, which serves purpose in the case of a "LOAD MORE COMMENTS" button with Ajax.
@paltman
Copy link
Member

paltman commented Jul 8, 2012

@tselaty Thanks for this patch. I think it looks good. Just want the chance to download and test it myself before I accept the pull request. Will do so this week hopefully.

@tselaty
Copy link
Author

tselaty commented Jul 8, 2012

No problem. I've made a customized version for an app I'm building which allows a limit, offer, and jQuery AJAX results to be pulled back. I may contribute the finalized changes once I've smoothed out the edges where it can be reusable in all cases.

Tim Selaty Jr.
TLS Web Solutions

On Saturday, July 7, 2012 at 8:48 PM, Patrick Altman wrote:

@tselaty Thanks for this patch. I think it looks good. Just want the chance to download and test it myself before I accept the pull request. Will do so this week hopefully.


Reply to this email directly or view it on GitHub:
#2 (comment)

@tselaty
Copy link
Author

tselaty commented Jul 8, 2012

To confirm, hold off on reviewing it just yet. I'll be contributing a much larger patch which includes default examples, usage documentation, and goodies for AJAX and general usage of the app.

Tim Selaty Jr.
TLS Web Solutions

On Sunday, July 8, 2012 at 2:31 AM, Tim Selaty Jr. wrote:

No problem. I've made a customized version for an app I'm building which allows a limit, offer, and jQuery AJAX results to be pulled back. I may contribute the finalized changes once I've smoothed out the edges where it can be reusable in all cases.

Tim Selaty Jr.
TLS Web Solutions

On Saturday, July 7, 2012 at 8:48 PM, Patrick Altman wrote:

@tselaty Thanks for this patch. I think it looks good. Just want the chance to download and test it myself before I accept the pull request. Will do so this week hopefully.


Reply to this email directly or view it on GitHub:
#2 (comment)

@paltman paltman closed this Jan 4, 2013
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.

None yet

2 participants