feature request: list-all action #25

Closed
mathowie opened this Issue Jun 23, 2015 · 6 comments

Comments

Projects
None yet
3 participants
@mathowie

I'd love it if there was a /gloss list-all feature that simply returned a comma delimited list of all defined words within glossary-bot.

@mathowie

This comment has been minimized.

Show comment
Hide comment
@mathowie

mathowie Jun 23, 2015

This would just be a tweak of the "learnings" command, just remove the date limit to get all records and order alphabetically.

This would just be a tweak of the "learnings" command, just remove the date limit to get all records and order alphabetically.

@tmaybe

This comment has been minimized.

Show comment
Hide comment
@tmaybe

tmaybe Jun 24, 2015

Collaborator

hmm...there's an undocumented random modifier to the learnings command (i.e. /gloss learnings random gets you 12 random selections from the database); it'd be easy to add an all as well.

Collaborator

tmaybe commented Jun 24, 2015

hmm...there's an undocumented random modifier to the learnings command (i.e. /gloss learnings random gets you 12 random selections from the database); it'd be easy to add an all as well.

@tmaybe

This comment has been minimized.

Show comment
Hide comment
@tmaybe

tmaybe Jul 1, 2015

Collaborator

This is now available as of 6ec9f58

You can now do the following:

  • /gloss learnings for the default 12 most recently defined terms
  • /gloss learnings random for a random 12 terms from the database
  • /gloss learnings all to see all the terms at once.
  • /gloss learnings 50 to see the 50 most recent terms.
  • /gloss learnings 50 10 to see 50 terms, offset by 10 (so, terms 10-60)
  • /gloss learnings today to see all the terms defined today
  • /gloss learnings yesterday to see the terms defined yesterday

And you can combine parameters in ways that make sense.

You may have problems upgrading because I added a database migration earlier this week. Follow the instructions here:

https://github.com/codeforamerica/glossary-bot#upgrade-on-heroku

or get in touch if you have any problems.

Collaborator

tmaybe commented Jul 1, 2015

This is now available as of 6ec9f58

You can now do the following:

  • /gloss learnings for the default 12 most recently defined terms
  • /gloss learnings random for a random 12 terms from the database
  • /gloss learnings all to see all the terms at once.
  • /gloss learnings 50 to see the 50 most recent terms.
  • /gloss learnings 50 10 to see 50 terms, offset by 10 (so, terms 10-60)
  • /gloss learnings today to see all the terms defined today
  • /gloss learnings yesterday to see the terms defined yesterday

And you can combine parameters in ways that make sense.

You may have problems upgrading because I added a database migration earlier this week. Follow the instructions here:

https://github.com/codeforamerica/glossary-bot#upgrade-on-heroku

or get in touch if you have any problems.

@tmaybe tmaybe closed this Jul 1, 2015

@tmaybe

This comment has been minimized.

Show comment
Hide comment
@tmaybe

tmaybe Jul 1, 2015

Collaborator

Whoops, I missed that you asked for alphabetical sort. That's now in as of 600be51

Use it like so: /gloss learnings alpha
So /gloss learnings all alpha will get you all records ordered alphabetically.

Collaborator

tmaybe commented Jul 1, 2015

Whoops, I missed that you asked for alphabetical sort. That's now in as of 600be51

Use it like so: /gloss learnings alpha
So /gloss learnings all alpha will get you all records ordered alphabetically.

@mathowie

This comment has been minimized.

Show comment
Hide comment
@mathowie

mathowie Jul 7, 2015

Thanks @tmaybe I got this all working for my team!

mathowie commented Jul 7, 2015

Thanks @tmaybe I got this all working for my team!

@migurski

This comment has been minimized.

Show comment
Hide comment

migurski commented Jul 7, 2015

bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment