Skip to content
This repository has been archived by the owner on Jul 17, 2020. It is now read-only.

Removed pagination from listcommands - print one multilined message inst... #154

Merged
merged 1 commit into from
Jan 11, 2014

Conversation

BobVul
Copy link
Contributor

@BobVul BobVul commented Dec 24, 2013

...ead

Rather than fiddling with multiple pages, just print all commands across multiple lines - the SE chat length limit is per-line, and we have practically a unlimited number of lines (pasted the 7k-line master.js with no problems).

Currently being tested on @allquixotic's fork in the RA chatroom.

@FirstWhack
Copy link
Contributor

This is useful for a number of things. I never knew it was per line.

On Dec 24, 2013, at 12:36 PM, Bob notifications@github.com wrote:

...ead

Rather than fiddling with multiple pages, just print all commands across multiple lines - the SE chat length limit is per-line, and we have practically a unlimited number of lines (pasted the 7k-line master.js with no problems).

Currently being tested on @allquixotic's fork in the RA chatroom.

You can merge this Pull Request by running

git pull https://github.com/Elusive138/SO-ChatBot Zirak
Or view, comment on, or merge it at:

#154

Commit Summary

Removed pagination from listcommands - print one multilined message instead
File Changes

M source/commands.js (29)
Patch Links:

https://github.com/Zirak/SO-ChatBot/pull/154.patch
https://github.com/Zirak/SO-ChatBot/pull/154.diff

@allquixotic
Copy link

+1. This has been tested and is working fine on Cavil in Root Access.

@Zirak
Copy link
Owner

Zirak commented Jan 10, 2014

Looks cool! Could you provide some links to usages?

@BobVul
Copy link
Contributor Author

BobVul commented Jan 10, 2014

Zirak pushed a commit that referenced this pull request Jan 11, 2014
@Zirak Zirak merged commit 4bd4b95 into Zirak:master Jan 11, 2014
@Zirak
Copy link
Owner

Zirak commented Jan 11, 2014

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

Successfully merging this pull request may close these issues.

None yet

4 participants